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

Update XSD to conform to style guide #85

Closed
cjerdonek opened this issue Apr 6, 2015 · 6 comments
Closed

Update XSD to conform to style guide #85

cjerdonek opened this issue Apr 6, 2015 · 6 comments
Milestone

Comments

@cjerdonek
Copy link
Contributor

This issue is to finish updating the schema to match our current style guidelines. Per this comment, I believe we're still in code freeze while @demcg works on this. Specifically, attribute and element names need to be updated (which was further clarified by issue #75).

@demcg
Copy link
Contributor

demcg commented Apr 6, 2015

@jungshadow @cjerdonek

I am continuing work on:

Naming Styles

  1. Attributes should be camelCased - The first letter of the second and subsequent words are capitalized.
  2. Elements should be TitleCased (PascalCased) - The first letter of every word is capitalized.
  3. Enumerations should be spinal-case - all lower case with spaces replaced with hyphens.

I will create a PR later today.

@jungshadow
Copy link
Collaborator

@demcg How's it coming? Any chance you might have a PR today? If not, just let us know how much longer you need. Also, if you need an additional set of hands, feel free to put up the branch and I can take over.

@jungshadow
Copy link
Collaborator

Well...that was timely :)

@demcg
Copy link
Contributor

demcg commented Apr 7, 2015

@jungshadow @cjerdonek
I noticed that we have a number of elements that are suffixed by "Url" that are of type xs:string ... should we change these to xs:anyURI?

created a issue about this. #87

demcg added a commit that referenced this issue Apr 7, 2015
Address Issue #85 -  Fixed case isssues in XSD
@demcg
Copy link
Contributor

demcg commented Apr 7, 2015

@cjerdonek Can I close this issue?

@cjerdonek
Copy link
Contributor Author

@demcg Yes, I think so. Thanks! We can always create a new issue if we see other stuff.

@demcg demcg closed this as completed Apr 7, 2015
@jdmgoogle jdmgoogle modified the milestone: Version 5.0 May 20, 2015
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

No branches or pull requests

4 participants