-
Notifications
You must be signed in to change notification settings - Fork 566
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
Creates a clean set of Pico examples #6800
Conversation
Disregard the changes under /pico as that will be merged in from #6801 and then we will rebase over that here. |
examples/pico/basics/src/main/java/io/helidon/examples/pico/basics/ToolBox.java
Outdated
Show resolved
Hide resolved
examples/pico/configdriven/src/main/java/io/helidon/examples/pico/configdriven/Main.java
Outdated
Show resolved
Hide resolved
examples/pico/interceptors/src/main/java/io/helidon/examples/pico/interceptors/Main.java
Outdated
Show resolved
Hide resolved
|
||
.level = INFO | ||
io.helidon.config.level = WARNING | ||
io.helidon.config.examples.level = FINEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this logging setting being used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this was a convention, looking at some other examples. It is not needed directly by my examples, no. Do you therefore suggest to remove them?
examples/pico/providers/src/main/java/io/helidon/examples/pico/providers/AngleGrinderSaw.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The pre-req changes have been extracted from this into its own PR.
Will resolve #6776