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

Configuration - Comments/Questions #150

Open
gsmet opened this issue May 21, 2024 · 1 comment
Open

Configuration - Comments/Questions #150

gsmet opened this issue May 21, 2024 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@gsmet
Copy link
Contributor

gsmet commented May 21, 2024

In https://redhat-developer-demos.github.io/quarkus-tutorial/quarkus-tutorial/03_configuration.html ,

  • I wonder if at the end of Adding a config property to application.properties, we should have a small sentence about live reload?
  • I think the note If you’re still running in live testing mode you will already see that the tests pass again and you can ignore the rest of this page and move on to the next step. should be made more prominent. And maybe also point to where you can see the Continuous Testing status (either in Dev UI or in the console)

I also wonder why we are talking about disabling continuous testing so soon? Because it kinda breaks the nice promise of never quitting live reload mode.

@kdubois kdubois added enhancement New feature or request good first issue Good for newcomers labels May 22, 2024
@kdubois
Copy link
Contributor

kdubois commented May 22, 2024

I agree, the tutorial should be updated to just run the tests with continuous testing & not interrupt the flow with killing dev mode, running the tests manually and then starting it up again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants