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

Added test for TOC to .travis.yml #205

Merged
merged 1 commit into from
Jan 14, 2018

Conversation

buslov
Copy link
Contributor

@buslov buslov commented Jan 11, 2018

Runs doctoc for README*.md.
If in newly doctoc'ed README*.md there are any diffs against ones commited in git HEAD then the test failes.
The script are using the latest anchor-markdown-header from repo to avoid the bug with non-ASCII headers.

.travis.yml Outdated
@@ -3,6 +3,12 @@ rvm:
- 2.2
before_script:
- gem install awesome_bot
- npm install doctoc
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a package.json file and add doctoc to it, and specify the new dep for anchor-markdown-header there? seems a bit cleaner than using these three commands, but that may just be me...

@RichardLitt
Copy link
Collaborator

What about a cleanup of node_modules and package-lock.json? Might make sense to either clean them up or add them to the .gitignore and maybe specify a package.json file.

Runs doctoc for README*.md.
If in newly doctoc'ed README*.md there are any diffs against
ones commited in git HEAD then the test failes.
The script are using the latest anchor-markdown-header from repo
to avoid the bug with non-ASCII headers.
@buslov
Copy link
Contributor Author

buslov commented Jan 13, 2018

@RichardLitt done

@RichardLitt RichardLitt merged commit 14ea082 into k88hudson:master Jan 14, 2018
@RichardLitt
Copy link
Collaborator

Thanks for this.

aroup pushed a commit to aroup/git-flight-rules that referenced this pull request Jan 17, 2020
Runs doctoc for README*.md.
If in newly doctoc'ed README*.md there are any diffs against
ones commited in git HEAD then the test failes.
The script are using the latest anchor-markdown-header from repo
to avoid the bug with non-ASCII headers.
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