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 #25

Merged
merged 7 commits into from
Oct 24, 2019
Merged

Clawdit #25

merged 7 commits into from
Oct 24, 2019

Conversation

seth-shaw-unlv
Copy link
Contributor

GitHub Issue: Islandora/documentation#1267

What does this Pull Request do?

Updates the conrtibuting.md to match the template and removes references to CLAW.

How should this be tested?

Preview and review.

Additional Notes:

There is still an open question about the README. It is currently structured as a tutorial. So, should we

  1. revise it to be consistent with the template but leave the bulk of it as-is; OR
  2. move the existing README content to a new TUTORIAL.md file and link it from a new README matching the template?

The last remaining CLAW references are to https://github.com/Islandora-Devops/migrate_7x_claw which needs to be declawed itself.

Interested parties

@manez, @Islandora/8-x-committers

@dannylamb
Copy link
Contributor

I planned on making the README match the others and pushing the whole tutorial over into documentation.

@seth-shaw-unlv
Copy link
Contributor Author

@manez & @dannylamb take a look and let me know what you think.

@manez
Copy link
Member

manez commented Oct 23, 2019

image

@seth-shaw-unlv
Copy link
Contributor Author

Thanks, @manez!

README.md Outdated Show resolved Hide resolved
@dannylamb dannylamb merged commit 3d5cdb7 into Islandora:dev Oct 24, 2019
hachacha pushed a commit to hachacha/migrate_islandora_csv that referenced this pull request Aug 5, 2020
* declaw

* add license

* mv readme to template and create new readme

* mascot

* remove travis badge

* fix badges

* simplify bages links
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