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

add notes about making minor model changes #1537

Merged
merged 3 commits into from
Sep 23, 2016

Conversation

mtbc
Copy link
Member

@mtbc mtbc commented Sep 22, 2016

Extends http://www.openmicroscopy.org/site/support/contributing-staging/data-model-schema.html to clarify how to adjust the process for "minor" model releases.

@hflynn hflynn added the develop label Sep 22, 2016
@hflynn
Copy link
Contributor

hflynn commented Sep 22, 2016

Staging build running now

Major or minor release?
^^^^^^^^^^^^^^^^^^^^^^^

A minor release of the OME model schema may suffice for changes like
Copy link
Member

Choose a reason for hiding this comment

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

Thanks a lot for that. As we are using may/must, I would propose to add the usual reference to RFC 2119 for the exact interpretation of the terms.

Copy link
Member Author

Choose a reason for hiding this comment

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

git grep 2119 in ome-documentation gives me zilch; could you point me to an example of what I should say? Do you want the all-caps style too?

Copy link
Member

Choose a reason for hiding this comment

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

Yes I do not think we have used it extensively across our documentation yet. An example of usage is at the top of ome/design#55.

Happy to discuss if this is still premature but we could certainly promote a more general usage /cc @hflynn @rleigh-codelibre

Copy link
Contributor

Choose a reason for hiding this comment

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

I am currently working on a BF docs PR which will also reference it so happy to have this become more widely used (it isn't referenced as yet but I think Roger has already used it for the version requirements page wording - I certainly have https://trello.com/c/HSPvUKRK/474-best-practice-for-imperatives-key-word-usage on the docs board after he pointed it out to me)

Copy link
Member Author

@mtbc mtbc Sep 22, 2016

Choose a reason for hiding this comment

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

I'm not sure they add much in this case (English "must" is already unequivocal even without the emphasis I added); I also wonder if those definitions should be a more general intro to the docs than repeated at the top of most pages, then the pages can just put them in some special style (like RFCs' all-caps)? Could certainly add here if seems warranted but I think this might be worth a deeper think beyond this PR.

Copy link
Member

Choose a reason for hiding this comment

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

Oh dear, didn't mean to open a can of worms :) Having this sentence as part of the introduction of the documentation would certainly be a sensible option especially if we reuse them more. Happy to use @hflynn's Trello card to channel this discussion.

@sbesson
Copy link
Member

sbesson commented Sep 23, 2016

Overall this reads well and describes most of the steps we have been through for 2016-06 version 2. Only minor question is whether adding a seealso note pointing at ome/bioformats#2553 for an example of minor schema release changes would be valuable here.

@dgault
Copy link
Member

dgault commented Sep 23, 2016

These changes look good and certainly make sense and cover any minor release differences that I can think of. It all reads well and is easy enough to understand from my perspective anyway.

@sbesson sbesson merged commit 33f86a5 into ome:develop Sep 23, 2016
@sbesson sbesson added this to the 5.3.0 milestone Nov 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants