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

Rename quarkus.test.native-image-profile to reflect its use for integration tests #29824

Closed
kdubb opened this issue Dec 13, 2022 · 2 comments · Fixed by #29826
Closed

Rename quarkus.test.native-image-profile to reflect its use for integration tests #29824

kdubb opened this issue Dec 13, 2022 · 2 comments · Fixed by #29826
Assignees
@geoand
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements area/testing
Milestone
2.16.0.CR1

Comments

@kdubb
Copy link
Contributor

kdubb commented Dec 13, 2022

Description

Since specific native image testing has been replaced by integration testing, which may or may not be a native image, this property name should reflect its new usage.

Implementation ideas

A name such as quarkus.test.integration-profile or similar would better reflect its current usage.
@kdubb kdubb added the area/housekeeping Issue type for generalized tasks not related to bugs or enhancements label Dec 13, 2022
@kdubb
Copy link
Contributor Author

kdubb commented Dec 13, 2022

/cc @geoand

@geoand
Copy link
Contributor

geoand commented Dec 13, 2022

Thanks for opening the issue!

@geoand geoand self-assigned this Dec 13, 2022
geoand added a commit to geoand/quarkus that referenced this issue Dec 13, 2022
@geoand
Introduce proper property name for controlling IT profile
15a6632 
Closes: quarkusio#29824
@geoand geoand mentioned this issue Dec 13, 2022
Merged
geoand added a commit to geoand/quarkus that referenced this issue Dec 13, 2022
@geoand
Introduce proper property name for controlling IT profile
b44310e 
Closes: quarkusio#29824
@quarkus-bot quarkus-bot bot added this to the 2.16 - main milestone Dec 13, 2022
geoand added a commit that referenced this issue Dec 13, 2022
@geoand
Merge pull request #29826 from geoand/#29824
0c62b1a 
Introduce proper property name for controlling IT profile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geoand geoand

Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements area/testing
Projects
None yet
Milestone
2.16.0.CR1
Development

Successfully merging a pull request may close this issue.

Introduce proper property name for controlling IT profile geoand/quarkus
2 participants
@kdubb @geoand