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

Remove "Atomic Batch Operations" #79

Merged
merged 3 commits into from
Mar 24, 2017
Merged

Remove "Atomic Batch Operations" #79

merged 3 commits into from
Mar 24, 2017

Conversation

ruebot
Copy link
Contributor

@ruebot ruebot commented Mar 23, 2017

@ruebot ruebot mentioned this pull request Mar 23, 2017
Closed
zimeon
zimeon reviewed Mar 23, 2017
View reviewed changes
index.html Outdated
<a href="#resource-versioning"></a>, <a href="#binary-fixity"></a>, <a href="#resource-authorization"></a>,
<a href="#atomic-operations"></a>, and <a href="#messaging"></a>.
<a href="#resource-versioning"></a>, <a href="#binary-fixity"></a>, <a href="#resource-authorization"></a>
, and <a href="#messaging"></a>.
Copy link
Contributor

Choose a reason for hiding this comment

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

This comma trailing after the preceding line will result in space before the comma

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, good catch!

@ruebot
Copy link
Contributor Author

ruebot commented Mar 23, 2017

I'm going to assume we're not enforcing column width on this file.

zimeon
zimeon approved these changes Mar 23, 2017
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.

👍

@awoods
Copy link
Collaborator

awoods commented Mar 23, 2017

This looks good. Let's give folks time to review by waiting on merge until tomorrow (Friday).

@awoods awoods merged commit 4053b68 into fcrepo:master Mar 24, 2017
@ruebot ruebot deleted the remove-abo branch March 24, 2017 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acoburn acoburn acoburn approved these changes

@zimeon zimeon zimeon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ruebot @awoods @zimeon @acoburn