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

Clawdit #62

Merged
merged 6 commits into from
Sep 27, 2019
Merged

Clawdit #62

merged 6 commits into from
Sep 27, 2019

Conversation

rangel35
Copy link

GitHub Issue: (Islandora/documentation#1267)

What does this Pull Request do?

Clean up of description entries in multiple files replacing Islandora Claw with Islandora 8 as appropriate.

What's new?

Update to readme to conform to new template, update maintainers.

  • Strictly a textual change to the description entries of different files, names, variables and urls left untouched

How should this be tested?

To be safe someone should probably do an install to make sure that no typos were added in the process of changing the descriptions

Interested parties

Tag (@Islandora-CLAW/committers)

@codecov
Copy link

codecov bot commented Sep 26, 2019

Codecov Report

Merging #62 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                dev      #62   +/-   ##
=========================================
  Coverage     84.89%   84.89%           
  Complexity       96       96           
=========================================
  Files            16       16           
  Lines           331      331           
  Branches          1        1           
=========================================
  Hits            281      281           
  Misses           49       49           
  Partials          1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 24c3450...4a345e7. Read the comment docs.

README.md Outdated Show resolved Hide resolved
@rangel35
Copy link
Author

@dannylamb removed playbook reference

@dannylamb
Copy link
Contributor

You can keep the part suggesting claw playbook. Just the second sentence needed to go.

removed too much before
@rangel35
Copy link
Author

@dannylamb I realized that and added it back in :)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@manez
Copy link
Member

manez commented Sep 26, 2019

Sorry! Late to the party on reviewing because I didn't notice I had not clicked the "Submit Review" button 😠

README.md Outdated Show resolved Hide resolved
@dannylamb dannylamb merged commit b0b4b81 into Islandora:dev Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants