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

Update README #1271

Merged
merged 8 commits into from
Jul 30, 2019
Merged

Update README #1271

merged 8 commits into from
Jul 30, 2019

Conversation

pyrmont
Copy link
Contributor

@pyrmont pyrmont commented Jul 23, 2019

This commit updates the content of the README. It is intended to be rebased against #1270 once that is merged.

@pyrmont pyrmont added needs-review The PR needs to be reviewed dependent-pr The PR is dependent on other changes being merged and will be reviewed later. labels Jul 23, 2019
This commit updates the content of the README.
@pyrmont pyrmont removed the dependent-pr The PR is dependent on other changes being merged and will be reviewed later. label Jul 24, 2019
@ashmaroli
Copy link
Contributor

Since you mention Jekyll, I think you should also mention kramdown (v2.0 uses Rouge as the default highlighter as well) and Asciidoctor..

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
pyrmont and others added 2 commits July 27, 2019 04:28
Co-Authored-By: Ashwin Maroli <ashmaroli@users.noreply.github.com>
README.md Show resolved Hide resolved
Copy link
Contributor

@robin850 robin850 left a comment

Choose a reason for hiding this comment

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

Great additions!

README.md Outdated Show resolved Hide resolved
@pyrmont pyrmont merged commit 85cf976 into rouge-ruby:master Jul 30, 2019
@pyrmont pyrmont removed the needs-review The PR needs to be reviewed label Jul 30, 2019
@pyrmont pyrmont deleted the docs.update-readme branch January 8, 2020 20:08
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

4 participants