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

mutable head extension #17

Merged
merged 11 commits into from Jan 27, 2021
Merged

mutable head extension #17

merged 11 commits into from Jan 27, 2021

Conversation

pwinckles
Copy link
Contributor

This is the initial draft of the mutable HEAD extension that Fedora 6 uses to create mutable HEADs in OCFL objects when Fedora's auto-versioning is disabled.

The oclf-java client, which is what Fedora 6 is using, currently implements this extension as written.

@rosy1280 rosy1280 added the draft label Aug 18, 2020
@pwinckles pwinckles added in-review and removed draft labels Aug 18, 2020
@pwinckles pwinckles changed the title inital draft of mutable head extension mutable head extension Aug 18, 2020
@rosy1280
Copy link
Contributor

@pwinckles can you resolve conflicts?

Copy link
Contributor

@bcail bcail left a comment

Choose a reason for hiding this comment

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

I don't plan to use mutable-head in our repo, but looks OK.

docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
bcail
bcail previously approved these changes Oct 20, 2020
@rosy1280 rosy1280 requested a review from bcail December 9, 2020 23:21
bcail
bcail previously approved these changes Dec 11, 2020
bcail
bcail previously approved these changes Dec 17, 2020
neilsjefferies
neilsjefferies previously approved these changes Jan 12, 2021
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Outdated Show resolved Hide resolved
docs/NNNN-mutable-head.md Show resolved Hide resolved
@pwinckles pwinckles dismissed stale reviews from neilsjefferies and bcail via 65a43d2 January 26, 2021 18:38
@awoods
Copy link
Member

awoods commented Jan 26, 2021

@bcail / @neilsjefferies : it appears that your "approvals" need to be refreshed based on additional commits to this PR.

@zimeon zimeon merged commit fe3965a into OCFL:master Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants