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

[WFCORE-4085] Temporarily use WF14 as the transformation target for EAP #3506

Merged
merged 1 commit into from Sep 6, 2018

Conversation

kabir
Copy link
Contributor

@kabir kabir commented Sep 5, 2018

7.2.0. Once EAP 7.2.0 is released there is a follow up task to switch it
to use EAP 7.2.0 proper in WFCORE-4089

https://issues.jboss.org/browse/WFCORE-4085

7.2.0. Once EAP 7.2.0 is released there is a follow up task to switch it
to use EAP 7.2.0 proper in WFCORE-4089

// TODO https://issues.jboss.org/browse/WFCORE-4089 Get rid of EAP_7_2_0_TEMP and enable EAP_7_2_0, once
// 7.2.0.GA-redhat-xx version is known
EAP_7_2_0_TEMP("11.0.0.Final", false, "14.0.0", "6.0.1.Final", "wf14"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't it be better to just call this 'EAP_7_2_0' - that way subsystems that reference it will not need to be updated when this gets renamed / a new one added.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@darranl I prefer a replace, for the reasons given in https://issues.jboss.org/browse/WFCORE-4085

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I suppose the Strings in this enum will still be updatable so as we have later tags we can still have CI runs against those tags.

If we have any further non Final tags will probably still be worth updating the referenced models so we don't end up in a position where we tag WF 15 and find it is incompatible with a later EAP tag.

@kabir kabir merged commit 86aec79 into wildfly:master Sep 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants