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

Print file name when searching for broken links #345

Closed
wants to merge 2 commits into from

Conversation

Sherlouk
Copy link
Contributor

@Sherlouk Sherlouk commented Dec 20, 2016

Changes in this pull request

Wasn't sure how to make it a one line wonder, so probably could be improved -- but slapped it into a while loops and print name before triggering the markdown-link-check.

Refers to #344 (comment)

Pull request checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I have reviewed the contributing guide

@Sherlouk
Copy link
Contributor Author

Sherlouk commented Dec 20, 2016

Investigating error - linting passed and, incorrectly, assumed it would check the shell scripts.

May be due to no semi colons (need to work on my shell 😅 )

@facebook-github-bot
Copy link
Contributor

@Sherlouk updated the pull request - view changes

@jessesquires
Copy link
Contributor

@Sherlouk awesome! 🎉 thanks!

@jessesquires
Copy link
Contributor

let's wait for ci to pass

@coveralls
Copy link

coveralls commented Dec 20, 2016

Coverage Status

Coverage remained the same at 97.955% when pulling 3dddca3 on Sherlouk:link-break into efca5a7 on Instagram:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.955% when pulling 3dddca3 on Sherlouk:link-break into efca5a7 on Instagram:master.

@Sherlouk
Copy link
Contributor Author

Passed 🎉

And who said I couldn't do shell? 😏

@facebook-github-bot
Copy link
Contributor

@jessesquires has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@jessesquires
Copy link
Contributor

@Sherlouk

@Sherlouk
Copy link
Contributor Author

Who? Me!?

@Sherlouk Sherlouk deleted the link-break branch December 20, 2016 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants