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

Ac update readme #367

Merged
merged 8 commits into from
Nov 7, 2017
Merged

Ac update readme #367

merged 8 commits into from
Nov 7, 2017

Conversation

Acribbs
Copy link
Member

@Acribbs Acribbs commented Nov 6, 2017

No description provided.

@sebastian-luna-valero
Copy link
Member

Thanks @Acribbs

I was also updating the README here:
https://github.com/CGATOxford/cgat/blob/SLV-update-install/README.rst

Please have a look and feel free to copy what you consider is missing.

@Acribbs
Copy link
Member Author

Acribbs commented Nov 6, 2017

@sebastian-luna-valero Thanks, I added the conda installation into the README and kept the pip but made a comment stating the recommended way of installation was conda. I have also updated the links to the documentation.

Updated links as per sebastians conversation
@Acribbs
Copy link
Member Author

Acribbs commented Nov 7, 2017

@sebastian-luna-valero I have updated the links, please could you make a check to make sure I have changed everything please

@sebastian-luna-valero
Copy link
Member

@Acribbs thanks!

@sebastian-luna-valero sebastian-luna-valero merged commit 3e1f5bf into master Nov 7, 2017
@sebastian-luna-valero sebastian-luna-valero deleted the AC-updateREADME branch November 7, 2017 10:44
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.

2 participants