Skip to content

Conversation

@yersan
Copy link
Contributor

@yersan yersan commented Jul 2, 2025

…sue: https://issues.redhat.com/browse/JBMETA-458

NOTE:
The current application-client_11.xsd included in this PR cannot be used to validate xml files due to a bug in this file.

I opened jakartaee/jakarta.ee#2199 to track it down, although I am not 100% sure if that repository is the correct one to fix these issues.

We can delay merging this , file a follow-up JBMETA Jira to address this issue once the changes have been fixed by Jakarta EE side, or temporarily fix it here, although I leave it as it is since this doesn't affect our tests.

@yersan yersan force-pushed the JBMETA-458 branch 2 times, most recently from fa857e8 to be3fcde Compare July 4, 2025 12:12
Copy link
Contributor

Choose a reason for hiding this comment

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

I filed https://issues.redhat.com/browse/JBMETA-465 related to this file. Doesn't need to be fixed to merge this.

~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
~ See the License for the specific language governing permissions and
~ limitations under the License.
-->
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use the standard JBoss Metadata header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
~ See the License for the specific language governing permissions and
~ limitations under the License.
-->
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use the standard JBoss Metadata header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
~ See the License for the specific language governing permissions and
~ limitations under the License.
-->
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use the standard JBoss Metadata header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@bstansberry bstansberry merged commit 6bfe07c into wildfly:main Jul 23, 2025
4 checks passed
@bstansberry
Copy link
Contributor

Thanks @yersan

@yersan yersan deleted the JBMETA-458 branch July 25, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants