Skip to content

Conversation

@cabler
Copy link

@cabler cabler commented Jul 11, 2016

I'm still trying to figure out how to do this... How do I change the link to https://github.com/DenefLab/flux-tools/blob/gh-pages/scripts/mothur.pbs ?

michberr added 16 commits May 25, 2016 17:48
* Created objectives for 5 lessons
* Created basic site infrastructure
and job syntax into different lessons

2) Renamed lessons

3) fix index.md to have paths to new lesson names
* Included a picture of globus
* Improved formatting
* Added a picture of a .bash_profile
* Improved formatting
* Added explanation of flux
* Changed names of lessons 5 and 6
* Removed lesson 06 and combined into 05 because of thematic similarity
* Added -j argument
* Updated to more recent mothur version
@michberr
Copy link
Contributor

Hey @cabler, two things:

  1. The lessons are all located in the lessons/ directory. It should be lesson 04. You will need to open the md file in RStudio, change the link, and then re-knit because the website renders the html file not the md. When you commit, there should therefore be two files with changes.

  2. You submitted this pull request to the master branch (default) rather than the gh-pages branch. For a tutorial style repo, whose main purpose is to serve as a website, i only commit to the gh-pages branch. It looks a little alarming when there's a pull request for 9,839 new lines of code and a few dozen files ;). From the command line you would git push origin gh-pages if you wanted to push to your version of gh-pages. I think you could also fix it when you you file the pull request by selecting your master branch but my gh-pages branch to merge.

@michberr michberr deployed to github-pages August 23, 2016 01:37 Active
@michberr michberr deployed to github-pages August 23, 2016 01:42 Active
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