Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Closed
ruebot opened this issue Feb 14, 2017 · 26 comments
Closed

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

ruebot opened this issue Feb 14, 2017 · 26 comments
Assignees

Comments

@ruebot
Copy link
Member

ruebot commented Feb 14, 2017

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
Copy link
Member Author

ruebot commented Feb 16, 2017

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
Copy link
Contributor

@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
Copy link
Member Author

ruebot commented Feb 17, 2017

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

@ruebot
Copy link
Member Author

ruebot commented Feb 19, 2017

@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
Copy link
Contributor

kimpham54 commented Feb 21, 2017

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

  • remove any links that point to 7.x-2.x, make them point to master or 8.x or what the 'master' branch would be for that repository
  • change any mention of "Islandora and Fedora project" to Islandora CLAW project
  • remove mentions or links to use cases that point to islandora-deprecated/Islandora-Fedora4-Interest-Group
  • make sure any anchor links refer to sections in the same repository, not back to the CLAW repository
  • Remove suggestion to use labels. Instead suggest that they specify Use Case|Documentation|Enhancement Request in the text or title.
  • generic welcome message: "If you are reading this document then you are interested in contributing to the Islandora CLAW. All contributions are welcome: use-cases, documentation, code, patches, bug reports, feature requests, etc. You do not need to be a programmer to speak up!"
  • remove mention of PSR-2, PSR-4 compliance
  • issues should be opened only in the Islandora-CLAW/CLAW repository

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
Copy link
Contributor

kimpham54 commented Feb 28, 2017

@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
Copy link
Member Author

ruebot commented Feb 28, 2017

  1. Yes. That will need to be left as is.

  2. Can you provide examples?

  3. Only applies to the CLAW repo.

@kimpham54
Copy link
Contributor

kimpham54 commented Feb 28, 2017

  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
Copy link
Member Author

ruebot commented Feb 28, 2017

@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
Copy link
Member

manez commented Feb 28, 2017

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
Copy link
Member Author

ruebot commented Mar 9, 2017

@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
Copy link
Member Author

ruebot commented Mar 16, 2017

@kimpham54 ping.

@kimpham54
Copy link
Contributor

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

@kimpham54
Copy link
Contributor

@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
Copy link
Member Author

ruebot commented Mar 20, 2017

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 added a commit to kimpham54/Alpaca that referenced this issue Mar 20, 2017
kimpham54 added a commit to kimpham54/Alpaca that referenced this issue Mar 20, 2017
@kimpham54
Copy link
Contributor

kimpham54 commented Mar 20, 2017

@ruebot @manez Islandora/Alpaca#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 added a commit to kimpham54/Alpaca that referenced this issue Mar 20, 2017
kimpham54 added a commit to kimpham54-archive/islandora_image that referenced this issue Mar 20, 2017
kimpham54 added a commit to kimpham54-archive/Syn that referenced this issue Mar 20, 2017
kimpham54 added a commit to kimpham54/drupal-project that referenced this issue Mar 20, 2017
ruebot pushed a commit to islandora-deprecated/drupal-project that referenced this issue Mar 20, 2017
ruebot pushed a commit to Islandora/Syn that referenced this issue Mar 20, 2017
kimpham54 added a commit to kimpham54/CLAW that referenced this issue Mar 20, 2017
kimpham54 added a commit to kimpham54/CLAW that referenced this issue Mar 20, 2017
ruebot pushed a commit that referenced this issue Mar 20, 2017
* address #537

* address #537
ruebot added a commit to yorkulibraries/claw_vagrant that referenced this issue Mar 21, 2017
@ruebot
Copy link
Member Author

ruebot commented Mar 21, 2017

PR: islandora-deprecated/claw_vagrant#23

dannylamb pushed a commit to islandora-deprecated/claw_vagrant that referenced this issue Mar 21, 2017
* Change default name, and bump memory
  * Partially addresses Islandora/documentation#537

* readme

* readme
ruebot pushed a commit to Islandora/jsonld that referenced this issue Mar 22, 2017
ruebot pushed a commit to islandora-deprecated/islandora_image that referenced this issue Mar 22, 2017
@ruebot
Copy link
Member Author

ruebot commented Mar 22, 2017

@kimpham54 those need to stay as is.

ruebot pushed a commit to yorkulibraries/drupal-project that referenced this issue Mar 22, 2017
@ruebot
Copy link
Member Author

ruebot commented Mar 23, 2017

@kimpham54 you can proceed with the Hypercube PR now.

dannylamb pushed a commit to islandora-deprecated/drupal-project that referenced this issue Mar 23, 2017
* Ignore web/sites/*/settings.php and web/sites/*/settings.local.php by default (drupal-composer#178)

* Prefix paths with a slash (drupal-composer#245)

* Update dev dependencies (SA-CORE-2017-001)

https://www.drupal.org/SA-2017-001

* Fix typo in README.md

Fixes drupal-composer#251

* Adding Islandora dependencies (#2)

* Adding Islandora dependencies

Part of Islandora/documentation#474

* Adding the github for claw-jsonld

* Removing requirement on islandora/islandora

islandora/islandora_collection requires it already.

* Removes repository/vcs entries from composer.json and adds requirements (#3)

* Removes repository/vcs entries from composer.json and adds requirements

* Removes islandora and claw-jsonld from composer.json requirements

* Revert to drupal/rdf_ui 1.x-dev version (#4)

* ISSUE-513

Fixes Drupal 8.1.x compatibility issue with drupal/rdf_ui by setting
version fixed to 1.x-dev

* Lock package versions to stuff we know work

* Adding Islandora dependencies (#2)

* Adding Islandora dependencies

Part of Islandora/documentation#474

* Adding the github for claw-jsonld

* Removing requirement on islandora/islandora

islandora/islandora_collection requires it already.

* Removes repository/vcs entries from composer.json and adds requirements (#3)

* Removes repository/vcs entries from composer.json and adds requirements

* Removes islandora and claw-jsonld from composer.json requirements

* Remove and ignore composer.lock; Resolves Islandora/documentation#530

* Pull Request Template (#8)

* address Islandora/documentation#537 (#9)
kimpham54 added a commit to kimpham54-archive/Hypercube that referenced this issue Mar 23, 2017
@kimpham54
Copy link
Contributor

@ruebot updated #537 (comment)

@ruebot
Copy link
Member Author

ruebot commented Mar 23, 2017

@kimpham54 thanks for all your work this!!

@ruebot ruebot closed this as completed Mar 23, 2017
@kimpham54
Copy link
Contributor

@ruebot thanks for putting up with my million fake PRs

dannylamb pushed a commit to dannylamb/drupal-project that referenced this issue May 2, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants