Skip to content

Table formatting and whitespace removal#177

Merged
wrightaprilm merged 17 commits intodatacarpentry:gh-pagesfrom
tmorrell:06formatting
Apr 12, 2017
Merged

Table formatting and whitespace removal#177
wrightaprilm merged 17 commits intodatacarpentry:gh-pagesfrom
tmorrell:06formatting

Conversation

@tmorrell
Copy link
Copy Markdown
Contributor

@tmorrell tmorrell commented Apr 7, 2017

Adding lesson 06 tables in markdown format (thanks to @rsdoiel and github.com/caltechlibrary/datatools) and cleaning up whitespace. Resolves #174

Copy link
Copy Markdown
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

Looks great @tmorrell.

@willingc
Copy link
Copy Markdown
Contributor

willingc commented Apr 8, 2017

@wrightaprilm This PR looks great to me. Double check the first plot which has no data plotted. I don't know enough about the lesson to know if there should be data points plotted. Other than that, I recommend merging.

Thanks @rsdoiel too 🍰

@tmorrell
Copy link
Copy Markdown
Contributor Author

tmorrell commented Apr 8, 2017

The first plot is supposed to be blank because there are no geom representations on the plot. I didn't edit course content in this PR-just formatting so the .md file is easier to work with. Thanks for the review!

@wrightaprilm
Copy link
Copy Markdown
Contributor

Looks very nice, merging now!

@wrightaprilm wrightaprilm reopened this Apr 12, 2017
@wrightaprilm wrightaprilm merged commit e2f46f6 into datacarpentry:gh-pages Apr 12, 2017
@tmorrell tmorrell deleted the 06formatting branch April 25, 2018 16:54
maxim-belkin pushed a commit that referenced this pull request May 9, 2018
zkamvar pushed a commit that referenced this pull request May 8, 2023
Table formatting and whitespace removal
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