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.

Sign up for GitHub

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

Jump to bottom

Create NNNN-fixed-length-n-tuple-trees.md #12

Closed
wants to merge 10 commits into from
Closed

Create NNNN-fixed-length-n-tuple-trees.md #12

wants to merge 10 commits into from

Conversation

neilsjefferies
Copy link
Member

First cut...enjoy.

@neilsjefferies neilsjefferies changed the title Fixed length n tuple tree extension Create 0002-fixed-length-n-tuple-trees.md May 20, 2020
pwinckles
pwinckles reviewed May 26, 2020
View reviewed changes
Copy link
Contributor

@pwinckles pwinckles left a comment

Choose a reason for hiding this comment

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

I like the changes that you made in this draft to make it clear the scope of this extension, ie fixed length, well-distributed identifiers only.

pwinckles
pwinckles reviewed Jun 8, 2020
View reviewed changes
This was referenced Jun 10, 2020
Merged
Closed
Closed
@rosy1280 rosy1280 removed the request for review from ahankinson August 18, 2020 13:45
@rosy1280 rosy1280 added the draft label Aug 18, 2020
awoods
awoods requested changes Sep 4, 2020
View reviewed changes
@pwinckles pwinckles mentioned this pull request Sep 9, 2020
Closed
@zimeon
Copy link
Contributor

zimeon commented Sep 15, 2020

0002 is now taken, this will have to be 0003 or later....

@zimeon zimeon changed the title Create 0002-fixed-length-n-tuple-trees.md Create NNNN-fixed-length-n-tuple-trees.md Sep 15, 2020
zimeon
zimeon requested changes Oct 14, 2020
View reviewed changes
@rosy1280 rosy1280 requested a review from zimeon October 20, 2020 14:00
@rosy1280
Copy link
Contributor

@neilsjefferies can you make the changes that have been requested to this PR?

Base automatically changed from master to main February 10, 2021 19:57
zimeon
zimeon requested changes May 4, 2021
View reviewed changes
Copy link
Contributor

@zimeon zimeon left a comment

Choose a reason for hiding this comment

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

Needs Extension Name: and OCFL Community Extensions Version: headers

@neilsjefferies
Copy link
Member Author

As per Editors Meeting 24 Aug 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zimeon zimeon zimeon requested changes

@rosy1280 rosy1280 Awaiting requested review from rosy1280

@julianmorley julianmorley Awaiting requested review from julianmorley

@awoods awoods Awaiting requested review from awoods

@bcail bcail Awaiting requested review from bcail

@pwinckles pwinckles Awaiting requested review from pwinckles

Assignees
No one assigned
Labels
in-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@neilsjefferies @zimeon @rosy1280 @awoods @pwinckles