Skip to content
This repository has been archived by the owner on Jan 9, 2019. It is now read-only.

Migrations: PRONOM 92 #63

Merged
merged 2 commits into from
Oct 28, 2017
Merged

Migrations: PRONOM 92 #63

merged 2 commits into from
Oct 28, 2017

Conversation

sevein
Copy link
Member

@sevein sevein commented Oct 19, 2017 •
edited
Loading

Other dependencies:

And #59 should be updated once this is merged and before we release.

@sevein sevein added the WIP label Oct 19, 2017
@sevein sevein self-assigned this Oct 19, 2017
@sevein sevein mentioned this pull request Oct 25, 2017
Closed
@sevein sevein removed the WIP label Oct 26, 2017
@sevein sevein requested a review from jrwdunham October 26, 2017 17:28
@sevein
Copy link
Member Author

sevein commented Oct 26, 2017

Two commits, but it'll be squashed.

@sevein
Copy link
Member Author

sevein commented Oct 26, 2017

Once this is merged we need to make a new released and have the dependency updated in the Dashboard.

@sevein sevein added this to the 1.7.2 milestone Oct 27, 2017
jrwdunham
jrwdunham approved these changes Oct 27, 2017
View reviewed changes
Copy link
Contributor

@jrwdunham jrwdunham left a comment

Choose a reason for hiding this comment

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

I deployed a local AM using dev/pronom-v92 and I sample-searched a bunch of the format versions, formats, id rules, normalization rules, etc. that I expected to find and found them.

@qubot qubot merged commit d6d0c4c into qa/1.x Oct 28, 2017
@qubot qubot deleted the dev/pronom-v92 branch October 28, 2017 05:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jrwdunham jrwdunham jrwdunham approved these changes

Assignees

@sevein sevein

Labels
None yet
Projects
None yet
Milestone
1.7.2
Development

Successfully merging this pull request may close these issues.
3 participants
@sevein @jrwdunham @qubot