-
Notifications
You must be signed in to change notification settings - Fork 14
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
Replicate specification of Object extension section #607
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@awoods leaving a comment because i'm not sure this is a hold up for approval. would rather others weight in.
draft/spec/index.md
Outdated
the purposes of extending the functionality of an OCFL Storage Root. The guidelines and limitations for the storage | ||
root `extensions` directory are defined in alignment with those of the [object extensions](#object-extensions). | ||
|
||
The `extensions` directory <span id="E112" class="rfc2119">MUST NOT</span> contain any files, and no sub-directories |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sentence is awkward. Why not:
The
extensions
directory MUST NOT contain any files or subdirectories other than extension subdirectories...
draft/spec/index.md
Outdated
other than extension sub-directories. Extension sub-directories <span id="W016" class="rfc2119">SHOULD</span> be named | ||
according to a <a>registered extension name</a>. | ||
|
||
> Non-normative note: Extension sub-directories should use the same name as a registered extension in order to both |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we run the two non-normative notes (this and the existing one that follows) into one? I.e. put a >
for the blank line between paras and remove the second "Non-normative note: "
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(otherwise looks good)
changes this to match other language changes in PR #607
changes this to match other language changes in PR #607
This update replicates the Object extension specification text from Object extension into Storage Root extension
Resolves: #557
Supersedes: #583