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

fix validation warnings in example inventories #48

Merged

julianmorley merged 1 commit into OCFL:main from pwinckles:validations

May 4, 2021

Contributor

pwinckles commented Apr 2, 2021

Adds missing mailto: to inventory examples so that the addresses are completely valid.


          fix validation warnings in example inventories

22064f1

zimeon approved these changes

View reviewed changes

Contributor

zimeon left a comment

I think this change is good and is reasonably considered "a simple fix" so can simply update the extension in place following:

Extensions are intended to be mostly static once published. Substantial revisions of content beyond simple fixes warrants publishing a new extension, and marking the old extension obsolete by updating the Obsoletes/Obsoleted by sections in each extension respectively. [https://github.com/OCFL/extensions]

rosy1280 requested review from rosy1280, awoods, bcail, julianmorley and neilsjefferies

May 4, 2021 13:54

rosy1280 approved these changes

View reviewed changes

julianmorley approved these changes

View reviewed changes

julianmorley merged commit e9d3e0d into OCFL:main

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

Reviewers

julianmorley julianmorley approved these changes

zimeon zimeon approved these changes

rosy1280 rosy1280 approved these changes

awoods Awaiting requested review from awoods

bcail Awaiting requested review from bcail

neilsjefferies Awaiting requested review from neilsjefferies

Labels

None yet