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

For Beginners document and reference in read me added #74

Merged
merged 16 commits into from
Feb 7, 2019

Conversation

rtilla1
Copy link
Contributor

@rtilla1 rtilla1 commented Aug 21, 2018

I added a relative link and reference to the "for beginners" documentation in the read.me file. I also added a step-by-step walk-through for turning any old machine into a CLAW environment.

Copy link
Member

@whikloj whikloj left a comment

Choose a reason for hiding this comment

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

One missing link, otherwise looks good.

forBeginners.md Outdated

>If you're not sure how much RAM and free memory on your hard drive is on the computer and you have a Windows computer, you can follow this [tutorial](https://www.computerhope.com/issues/ch000149.htm).

>If you don't have an operating system running, you can try to install the Ubuntu operating system below and follow these [steps](**return here to include link**)
Copy link
Member

Choose a reason for hiding this comment

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

I think you forgot to include a link 😉

forBeginners.md Outdated
vagrant up
On your new Ubuntu computer, download the following:
1. [Virtual Box](https://www.virtualbox.org/)
2. [Vagrant](https://www.vagrantup.com/) (version 2.0 or required)

Choose a reason for hiding this comment

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

"Version 2.0 or ??? required" -- OR -- "is version 2.0 is required"? Otherwise this is great!

@rtilla1
Copy link
Contributor Author

rtilla1 commented Sep 19, 2018

Sorry...I had to do quite a bit of research and editing to correct this missing link. I believe it's corrected now.

@whikloj
Copy link
Member

whikloj commented Sep 19, 2018

I haven't actually run through this process, but it all looks correct and if we find some corrections later we can handle them then.
👍

@whikloj
Copy link
Member

whikloj commented Sep 19, 2018

@rtilla1 this is good, but I couldn't find your name (or Tulane) on the list of current CLAs.
https://github.com/Islandora/islandora/wiki/Contributor-License-Agreements

Have you signed an individual CLA or has Tulane submitted one? We'll need one before I can merge this, my apologizes I should have caught this earlier.

@rtilla1
Copy link
Contributor Author

rtilla1 commented Sep 21, 2018 via email

@whikloj
Copy link
Member

whikloj commented Jan 17, 2019

@rtilla1 bump... did you get anywhere with Tulane or would you be able to sign a iCLA?

@whikloj
Copy link
Member

whikloj commented Feb 7, 2019

@manez @dannylamb what do we do here?

We had a similar situation in Fedora where someone had come and helped update a bunch of dependency versions (so no code changes) and disappeared. We merged it with the plan to add a note to the CLA table indicating this deviance from the norm.

Or we dump it 🤷‍♂️

@manez
Copy link
Contributor

manez commented Feb 7, 2019

@rtilla1 is certainly still around, but might not be getting her GitHub notices. I'll email her off-thread and see if she's made any CLA progress.

@manez
Copy link
Contributor

manez commented Feb 7, 2019

And it's now official! We have a cCLA from Tulane and @rtilla1 's awesome work is good to go 🎉 🎉 🎉

ETA: ping @whikloj

@whikloj whikloj merged commit 27fce05 into Islandora-Devops:master Feb 7, 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.

5 participants