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

minor edit #303

Closed
wants to merge 1 commit into from
Closed

minor edit #303

wants to merge 1 commit into from

Conversation

paddytobias
Copy link

We'll was spelt we???ll. Made the fix.

We'll was spelt we???ll. Made the fix.
@paddytobias paddytobias changed the title spelling mistake minor edit Oct 26, 2017
@colinmorris colinmorris mentioned this pull request Nov 5, 2017
@katrinleinweber
Copy link
Contributor

Hello and thanks for this update suggestion! I'm went through some older PRs, and #293 fixed the same typo. However, not in the .Rmd file as well. I can still merge your PR, if you update it with that :-)

Copy link
Contributor

@katrinleinweber katrinleinweber left a comment

Choose a reason for hiding this comment

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

Hi @paddytobias! I'm very sorry that your PR wasn't picked up for so long. Are you still interested, though? If yes, please add a commit that reflects your fix in the .Rmd as well, because that is the source for the .md file. Cheers :-)

@katrinleinweber
Copy link
Contributor

Not sure when, but this has been fixed already: https://swcarpentry.github.io/r-novice-inflammation/aio/

fmichonneau pushed a commit that referenced this pull request Sep 11, 2018
links.md: add links to carpentries' lessons overview pages
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.

None yet

2 participants