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-5640][WFCORE-5641] XSD and XML cleanup #4814

Merged

Conversation

jmesnil
Copy link
Member

@jmesnil jmesnil commented Oct 18, 2021

This PR includes:

  • [WFCORE-5640] Relax constraint on host-releaseType's id attribute
    Instead of making it an enum that must be updated with every WildFly
    Core release, let's use a string type.

    JIRA: https://issues.redhat.com/browse/WFCORE-5640

  • [WFCORE-5641] Clean up DomainXml parser classes
    Remove all WildFly core Xml classes that were identical to their previous
    versions.
    This only leaves classes that introduced actual changes to the
    XML parsing

    1 commit per XML class

    JIRA: https://issues.redhat.com/browse/WFCORE-5641

@jmesnil jmesnil requested a review from yersan October 18, 2021 07:29
Instead of making it an enum that must be updated with every WildFly
Core release, let's use a string type.

JIRA: https://issues.redhat.com/browse/WFCORE-5640

Signed-off-by: Jeff Mesnil <jmesnil@redhat.com>
Remove all DomainXml classes that were identical to their previous
versions.
This only leaves DomainXml classes that introduced actual changes to the
XML parsing

JIRA: https://issues.redhat.com/browse/WFCORE-5641

Signed-off-by: Jeff Mesnil <jmesnil@redhat.com>
Remove all StandaloneXml classes that were identical to their previous
versions.
This only leaves StandaloneXml classes that introduced actual changes to
the
XML parsing

JIRA: https://issues.redhat.com/browse/WFCORE-5641

Signed-off-by: Jeff Mesnil <jmesnil@redhat.com>
Remove all HostXml classes that were identical to their previous
versions.
This only leaves HostXml classes that introduced actual changes to the
XML parsing

JIRA: https://issues.redhat.com/browse/WFCORE-5641

Signed-off-by: Jeff Mesnil <jmesnil@redhat.com>
@jmesnil jmesnil force-pushed the WFCORE-5640_release_host-release_id_restriction branch from 567d42f to aa0fdfb Compare October 18, 2021 07:31
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Oct 18, 2021
@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Oct 19, 2021
@jmesnil jmesnil merged commit c3e1354 into wildfly:main Oct 19, 2021
@jmesnil
Copy link
Member Author

jmesnil commented Oct 19, 2021

@yersan thanks for the review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants