-
Notifications
You must be signed in to change notification settings - Fork 49
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 the XML format specification in Appendix B to provide a robust link to the XML schema file #516
Comments
Dear @larsbarring I agree that putting the location of the Cheers Jonathan |
OK to treat it as an enhancement, in particular as some of the associated issues are enhancements. |
I have now verified that providing a uri (i.e. a web link to an xsd file) works as expected (see cf-convention/cf-convention.github.io#470)) , although of course not with the specific link suggested here. |
As this a proposal for an As this is a small and rather technical change, but a key piece in the string of issues outlined in cf-convention/cf-convention.github.io#457 it would be much appreciated if this issue would not be left hanging. May I be so bold as to ping @sadielbartholomew, @davidhassell for comments. Many thanks, |
I have read the proposal, after prompting by Lars above, and I agree that this is an enhancement, both in terms of categorisation of the Issue and in terms of actively improving the CF Conventions, in particular the infrastructure in this case. I say the latter because being able to easily access the XML schema file is clearly a benefit over it being hard to locate and/or get to, as has correctly been identified is true at present. So I, for one, support this. |
With this I think that we can start the three-week count down, meaning that the associated PR can be merged on April, 17, if the preceding PR cf-convention/cf-convention.github.io#468 has been merged. |
@sadielbartholomew: May I once again call on you to check the PR #517 and merge if you are satisfied. In particular check lines 17-18 so that I got the terminology right. Many thanks, |
Title
Update the XML format specification in Appendix B to provide a robust link to the XML schema file
Moderator
Not yet
Requirement Summary
If a XML schema definition file (.xsd) is specified in the XML file it has to be available to the software reading the XML file.
Technical Proposal Summary
This is one in a string of issues that aims to improve the format of the XML version of the standard name table files, see cf-convention/cf-convention.github.io#457 for background and overview.
Specifically, this proposal updates Appendix B to include the the robust link implemented through issue cf-convention/cf-convention.github.io#469. It also updates
Example B.1
to account for the recent changes.Benefits
All users of the XML version of the standard name table will be able to more effectively draw on the format specification laid out in Appendix B.
Status Quo
This proposal have no bearing on the material content of the CF Conventions.
Associated pull request
#517
Detailed Proposal
Only a small technical change is required:
Current text:
Note that the boldfaced filename does not have directory specified or is a html link, which means that there is no information as to how it can be reached. This will be replaced by the link resulting from implementation of cf-convention/cf-convention.github.io#469. Based on how other links to are presented on the same web page is is possible to preliminary infer how the link will look like, which is what is used in the draft PR.
The text was updated successfully, but these errors were encountered: