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

WFLY-13176 Add jboss-ejb-client_1_3.xsd schema #13151

Merged
merged 1 commit into from Apr 4, 2020
Merged

WFLY-13176 Add jboss-ejb-client_1_3.xsd schema #13151

merged 1 commit into from Apr 4, 2020

Conversation

RanabirChakraborty
Copy link
Contributor

@RanabirChakraborty
Copy link
Contributor Author

I have added a jboss-ejb-client_1_3.xsd schema. Please review and let me know what you think ! :)

@wildfly-ci wildfly-ci added the deps-ok Dependencies have been checked, and there are no significant changes label Mar 26, 2020
</xsd:annotation>
</xsd:attribute>
</xsd:complexType>

Copy link
Contributor

Choose a reason for hiding this comment

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

need to fix the indentation, eg. replace tab with space consistent with other elements.

<xsd:annotation>
<xsd:documentation>
Cluster configurations applicable for this client context
</xsd:documentation>
Copy link
Contributor

Choose a reason for hiding this comment

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

need to change documentation to reflect the purpose of the new element. Probably something like Remoten connection profile selected for this client context.

@chengfang
Copy link
Contributor

@tadamski please review

profile element and profileType element definition are the only additon compared to jboss-ejb-client 1.2.


<!--
~ JBoss, Home of Professional Open Source.
~ Copyright 2012, Red Hat, Inc., and individual contributors
Copy link
Member

Choose a reason for hiding this comment

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

My watch says its 2020.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Previously I thought I would change it to 2020 but I was not sure about how the copyright thing works.You can check now, I have made the changes. :)

@bstansberry bstansberry added the ready-for-merge Only for use by those with merge permissions! label Apr 3, 2020
@bstansberry bstansberry merged commit 32e063c into wildfly:master Apr 4, 2020
@bstansberry
Copy link
Contributor

Thanks!

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 Only for use by those with merge permissions!
Projects
None yet
6 participants