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

Verify warning log message when a build time property is updated #718

Closed
wants to merge 1 commit into from

Conversation

pjgg
Copy link
Contributor

@pjgg pjgg commented Jun 15, 2022

Summary

Related to this upstream commit: quarkusio/quarkus#25512

Please select the relevant options.

  • New scenario (non-breaking change which adds functionality)

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@pjgg pjgg added the triage/backport-2.7? It needs to backport changes to branch 2.7 label Jun 15, 2022
@pjgg pjgg requested a review from jsmrcka June 15, 2022 10:31
@pjgg pjgg force-pushed the feat/warning_log branch from b7ae3eb to 1bc8a90 Compare June 15, 2022 11:22
@pjgg pjgg force-pushed the feat/warning_log branch from 1bc8a90 to 0276ac9 Compare June 15, 2022 12:01
@pjgg pjgg requested a review from jsmrcka June 15, 2022 12:04
@pjgg pjgg marked this pull request as draft June 15, 2022 12:33
@pjgg
Copy link
Contributor Author

pjgg commented Jun 15, 2022

Keep on hold this PR until this issue get clarified

@pjgg
Copy link
Contributor Author

pjgg commented Jun 27, 2022

This PR is not going to be merged, seems an issue in upstream that is already open

@pjgg pjgg closed this Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DON'T MERGE triage/backport-2.7? It needs to backport changes to branch 2.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants