Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend Quarkus live reload test #85

Merged
merged 1 commit into from
May 20, 2021

Conversation

jsmrcka
Copy link
Contributor

@jsmrcka jsmrcka commented May 18, 2021

In reload test, in addition to modifying an existing class,
add a new class and verify that it has been loaded.

Covers https://issues.redhat.com/browse/QUARKUS-1020.

@kshpak
Copy link
Contributor

kshpak commented May 18, 2021

Working on it

@jsmrcka jsmrcka force-pushed the live-reload-add-class-td branch from 9af89c1 to dd83c74 Compare May 18, 2021 11:16
In reload test, in addition to modifying an existing class,
add a new class and verify that it has been loaded.

Covers https://issues.redhat.com/browse/QUARKUS-1020.
@jsmrcka jsmrcka force-pushed the live-reload-add-class-td branch from dd83c74 to a8e0634 Compare May 18, 2021 11:19
@jsmrcka
Copy link
Contributor Author

jsmrcka commented May 18, 2021

I force-pushed some changes to READMEs, which I forgot to include before.

Copy link
Contributor

@kshpak kshpak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, let's see what @rsvoboda will say about new file in directory vs. new file by copying existing one and changing the endpoint

@rsvoboda rsvoboda merged commit ddcf9f3 into quarkus-qe:main May 20, 2021
@jsmrcka jsmrcka deleted the live-reload-add-class-td branch June 9, 2021 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants