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.

Sign up for GitHub

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

Jump to bottom

Remove config customizer #5332

Closed
dmlloyd opened this issue Nov 8, 2019 · 2 comments · Fixed by #22787
Closed

Remove config customizer #5332

dmlloyd opened this issue Nov 8, 2019 · 2 comments · Fixed by #22787
Milestone
2.7.0.CR1

Comments

@dmlloyd
Copy link
Member

dmlloyd commented Nov 8, 2019

We should probably remove config customizer. It's better to produce a config source instead, and simplifies the native image stuff and the config handling code (not to mention all the APIs in between).
@stuartwdouglas
Copy link
Member

@dmlloyd is this still an issue? I am not sure what it is referring too?

@dmlloyd
Copy link
Member Author

dmlloyd commented Aug 24, 2021

It was referring to anything which depends on io.quarkus.deployment.QuarkusAugmentor#configCustomizer.

geoand added a commit to geoand/quarkus that referenced this issue Jan 11, 2022
@geoand
Remove config customizer
2fb4c7f 
This is no longer used anywhere, so let's remove it

Resolves: quarkusio#5332
@geoand geoand mentioned this issue Jan 11, 2022
Merged
geoand added a commit to geoand/quarkus that referenced this issue Jan 11, 2022
@geoand
Remove config customizer
ad6b2d5 
This is no longer used anywhere, so let's remove it

Resolves: quarkusio#5332
gsmet added a commit that referenced this issue Jan 11, 2022
@gsmet
Merge pull request #22787 from geoand/#5332
91ce7e2 
Remove config customizer
@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Jan 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.7.0.CR1
Development

Successfully merging a pull request may close this issue.

Remove config customizer geoand/quarkus
2 participants
@dmlloyd @stuartwdouglas