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 sub-resolution proposal #83

Merged
merged 1 commit into from
Feb 7, 2018
Merged

Add sub-resolution proposal #83

merged 1 commit into from
Feb 7, 2018

Conversation

rleigh-codelibre
Copy link
Contributor

No description provided.

@joshmoore
Copy link
Member

No objections from my side to starting here as a v1. (I also like the subdirectories, but that's one we should agree on since renames won't be redirected as with full repositories.)

@sbesson
Copy link
Member

sbesson commented Feb 7, 2018

Agreed with regard to directories, I can see 2 options:

  • YYYY/MM/<proposal>.md as suggested here
  • nnn/<proposal>.md where nnn is an incremental number

As discussed recently, slight personal preference here for option 2 personally in the anticipation this could eventually turn into something more official like PEP and JEP where a attribution number helps with the naming. In this case, the creation date could still be captured as internal metadata.

@rleigh-codelibre
Copy link
Contributor Author

If you want to merge this, I could follow up with a PR which moves every existing proposal to a versioned directory nnn/....

@sbesson
Copy link
Member

sbesson commented Feb 7, 2018

Sounds good. In terms of numbering, we can probably call this one 005. Looking at the history, that would give:

Index Proposal
001 cli_tree.md
002 omero_features.md
003 ansible-omero-roles.md
004 tiling_scoping.md
005 tiff_sub_resolution_support.md

Starting the indexing of concrete proposals at 1 so that anyone can use the 0 for the indexing as a RFE.

@sbesson sbesson merged commit 8ec08fe into ome:master Feb 7, 2018
@rleigh-codelibre rleigh-codelibre deleted the sub-resolutions branch February 7, 2018 17:55
@rleigh-codelibre
Copy link
Contributor Author

#84 is the renumbering work.

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.

3 participants