Islandora / documentation

Contains islandora's documentation and main issue queue.
MIT License
104 stars 71 forks source link

Documentation audit for 7.x-2.x #537

Closed ruebot closed 7 years ago

ruebot commented 7 years ago

We need to audit the docs directory for any mention of 7.x-2.x, and if it exists replace it with Islandora CLAW.

ruebot commented 7 years ago

Here is some help if somebody wants to tackle it:

[nruest@gorila:docs] (git)-[master]-$ grep -R "7.x-2.x" .                    
./technical-documentation/docs-build.md:This documentation is built using [mkdocs](http://www.mkdocs.org/), a static site generator that is geared towards building project documentation. The documentation is created in the [Markdown](http://en.wikipedia.org/wiki/Markdown) format, and it all resides in the [`docs`](https://github.com/Islandora-Labs/islandora/tree/7.x-2.x/docs) directory in the repository. The organization of the documenation is controlled by the [`mkdocs.yml`](https://github.com/Islandora-Labs/islandora/blob/7.x-2.x/mkdocs.yml) in the root of the repository.
./user-documentation/intro-to-claw.md:Islandora CLAW is the project name for development of Islandora to work with Fedora 4, also known as Islandora 7.x-2.x. To fully understand Islandora CLAW, it is best to start by looking at its contrasts to the previous version of Islandora, known as 7.x-1.x. 
./user-documentation/intro-to-claw.md:Islandora CLAW (or Islandora 7.x-2.x) does more than simply replace that base layer with Fedora 4. It is a total re-architecting of the interaction between the various pieces. Rather than a hamburger, Islandora CLAW is a chimera:
./contributing/templates/committerAnnounce.txt:The Islandora 7.x-2.x Project committers have asked [New Committer] to become a committer and we are pleased to announce that they have accepted.
./contributing/templates/committerInvite.txt:(on behalf of the Islandora 7.x-2.x Committers)
./contributing/CONTRIBUTING.md:That is great! In this case please send us a pull request as described in section _[Create a pull request](https://github.com/Islandora-CLAW/CLAW/blob/7.x-2.x/CONTRIBUTING.md#create-a-pull-request)_ below.
./contributing/CONTRIBUTING.md:1. [Fork](https://help.github.com/articles/fork-a-repo) the Islandora GitHub repository at [https://github.com/Islandora-CLAW/CLAW](https://github.com/Islandora-CLAW/CLAW) to your personal GitHub account. If you already have a fork of [https://github.com/Islandora/islandora](https://github.com/Islandora/islandora) Github will prevent you from creating a new fork; in such a case you can continue to use [https://github.com/Islandora/islandora](https://github.com/Islandora/islandora) to issue pull request, be cautious of which branches you work from though (you'll want to base your work off of 7.x-2.x). Also be cautious of which repository your issuing your Pull Request to (you'll want to issue your pull request to [https://github.com/Islandora-CLAW/CLAW](https://github.com/Islandora-CLAW/CLAW)). See [Fork a repo](https://help.github.com/articles/fork-a-repo) for detailed instructions.
kimpham54 commented 7 years ago

@ruebot some repositories have 8.x.-1.x, master, or both branches. In the case where there are both branches, should I mention that in contributing.md to base your work off of master or off of 8.x-1.x?

ruebot commented 7 years ago

@kimpham54 just list any repo that you have a question about, and we address them one-by-one.

ruebot commented 7 years ago

@kimpham54 can you gather links to all your pull requests here in a comment, and let us know when you're ready for us to review them all. You can edit the comment if need be.

kimpham54 commented 7 years ago

cleanup tasks for CONTRIBUTING.md (will continue to be updated):

additional notes when auditing documentation: add a contributing.md if it doesn't exist to every repository work on separate branch reference the original issue in the ticket add all links in one comment for review

kimpham54 commented 7 years ago

@ruebot

  1. Does islandora-7x-2x-committers@googlegroups.com still exist? What should the contact be here: https://github.com/Islandora-CLAW/CLAW/blob/master/docs/contributing/templates/closeCommitterVote.txt#L1

  2. Should we have the same CONTRIBUTING.md wording in every repository with the appropriate links/reference to branches modified? I will still look at each CONTRIBUTING.md and note at any major discrepancies but will eventually replace them with the same evolving template.

  3. Is this still true:

Fork the Islandora GitHub repository at https://github.com/Islandora-CLAW/CLAW to your personal GitHub account. If you already have a fork of https://github.com/Islandora/islandora Github will prevent you from creating a new fork; in such a case you can continue to use https://github.com/Islandora/islandora to issue pull request, be cautious of which branches you work from though (you'll want to base your work off of 7.x-2.x). Also be cautious of which repository your issuing your Pull Request to (you'll want to issue your pull request to https://github.com/Islandora-CLAW/CLAW). See Fork a repo for detailed instructions.

I'm guessing that if it is, this does not apply to every repository but only the CLAW repo?

From https://github.com/Islandora-CLAW/CLAW/blob/master/CONTRIBUTING.md#create-a-pull-request.

ruebot commented 7 years ago
  1. Yes. That will need to be left as is.

  2. Can you provide examples?

  3. Only applies to the CLAW repo.

kimpham54 commented 7 years ago
  1. Example template: https://github.com/Islandora-CLAW/Alpaca/pull/33/files would be the template for CONTRIBUTING.md in each repository that isn't CLAW, with the only changes that need to be made for each document under Create a pull request:
  1. Fork this repository at CHANGEME to your personal GitHub account.Be cautious of which branches you work from though (you'll want to base your work off of CHANGEME:MASTER/8.X). See Fork a repo for detailed instructions.
ruebot commented 7 years ago

@kimpham54 I'm fine with that. But, this might become a gotcha in the future when updating the document again. @manez, what do you think?

manez commented 7 years ago

I'd prefer to keep it generic if there's any way we can rephrase that. Just looking at how much variation has crept into something as simple as the LICENCE file on the 7.x stack, which is supposed to be identical, the fewer moving parts we maintain in these every-repo-has-one docs, the better.

ruebot commented 7 years ago

@kimpham54 we had some Islandora-CLAW GitHub org changes over the last week, so there should be less work here. How are things going on your end with this ticket?

ruebot commented 7 years ago

@kimpham54 ping.

kimpham54 commented 7 years ago

@ruebot sorry, just got back from vacation and didn't have my laptop. i'll be sending PRs shortly

kimpham54 commented 7 years ago

@ruebot @manez trying to describe forking a repository in a generic way:

  1. Fork this repository to your personal GitHub account. Be cautious of which branches you work from though (you'll want to base your work off of the the most current stable branch - don't be afraid to ask if you're unsure which one it is). See Fork a repo for detailed instructions.

The repositories in Islandora-CLAW currently use either master or 8.x-1.x.

ruebot commented 7 years ago

Fork this repository to your personal GitHub account.

That can be a personal or institutional depending on CLAs.

You could just say something along the lines of making sure your work begins with syncing with the remote and link to this: https://help.github.com/articles/syncing-a-fork/

The repositories in Islandora-CLAW currently use either master or 8.x-1.x.

All Drupal modules use the Drupal version branch convention. Everything else is master.

kimpham54 commented 7 years ago

@ruebot @manez https://github.com/Islandora-CLAW/Alpaca/pull/37 is ready for review. I've closed all other PRs for now. If the CONTRIBUTING.md looks good, I will start creating pull requests for the other repositories with the same CONTRIBUTING.md

kimpham54 commented 7 years ago

@ruebot @manez Updated Islandora-CLAW/Alpaca#37 - please review

ruebot commented 7 years ago

Resolved with:

kimpham54 commented 7 years ago

PRs to review:

ruebot commented 7 years ago

PR: https://github.com/Islandora-CLAW/claw_vagrant/pull/23

kimpham54 commented 7 years ago

@ruebot none of the repositories now mention 7.x-2.x, with the exception of https://github.com/Islandora-CLAW/CLAW/blob/master/docs/contributing/templates/closeCommitterVote.txt#L1 and https://github.com/Islandora-CLAW/CLAW/blob/master/docs/contributing/templates/committerVote.txt#L1

ruebot commented 7 years ago

@kimpham54 those need to stay as is.

ruebot commented 7 years ago

@kimpham54 you can proceed with the Hypercube PR now.

kimpham54 commented 7 years ago

@ruebot updated https://github.com/Islandora-CLAW/CLAW/issues/537#issuecomment-287839669

ruebot commented 7 years ago

@kimpham54 thanks for all your work this!!

kimpham54 commented 7 years ago

@ruebot thanks for putting up with my million fake PRs