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.

Sign up for GitHub

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

Jump to bottom

html-proofer linting: Blog Posts, Related Tools #45

Merged
merged 9 commits into from
Oct 14, 2017
Merged

html-proofer linting: Blog Posts, Related Tools #45

merged 9 commits into from
Oct 14, 2017

Conversation

olleolleolle
Copy link
Contributor

@olleolleolle olleolleolle commented Oct 14, 2017 •
edited
Loading

This PR updates two articles, Blog Posts and Related Tools.

  • Repaired markup to be Markdown (links, table, local linking)
  • Update hyperlinks to use the most accurate content and the fastest way to get there (avoiding redirects, for example)
  • Blog Posts: go through content, make our language more neutral
  • Related Tools: go through all old content, get rid of things lost to time

Preview links: Blog Posts and Related Tools

Fixes #38

@olleolleolle olleolleolle changed the title Blog-posts: cucumber is hot tech at 2009 RailsConf html-proofer linting: Blog-posts: cucumber is hot tech at 2009 RailsConf Oct 14, 2017
@olleolleolle olleolleolle changed the title html-proofer linting: Blog-posts: cucumber is hot tech at 2009 RailsConf html-proofer linting: Blog-posts: "cucumber is hot tech at 2009 RailsConf" article moved Oct 14, 2017
@olleolleolle olleolleolle changed the title html-proofer linting: Blog-posts: "cucumber is hot tech at 2009 RailsConf" article moved html-proofer linting: content/about/blog-posts.md Oct 14, 2017
@olleolleolle olleolleolle changed the title html-proofer linting: content/about/blog-posts.md html-proofer linting: Blog Posts, Related Tools Oct 14, 2017
@olleolleolle olleolleolle self-assigned this Oct 14, 2017
mxygem
mxygem approved these changes Oct 14, 2017
View reviewed changes
Copy link
Member

@mxygem mxygem left a comment

Choose a reason for hiding this comment

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

LGTM!

I initially commented on the inclusion of the C++ repo but then I realized it's not a native implementation of Cucumber.

@olleolleolle olleolleolle merged commit b8e1e49 into master Oct 14, 2017
@olleolleolle olleolleolle deleted the fix/link-to-infoworld-article-moved branch October 14, 2017 20:09
@mlvandijk mlvandijk mentioned this pull request Oct 15, 2017
Closed
16 tasks
@plaindocs
Copy link
Contributor

Awesome stuff!

@olleolleolle
Copy link
Contributor Author

@plaindocs Thanks, you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mxygem mxygem mxygem approved these changes

@plaindocs plaindocs Awaiting requested review from plaindocs

@mlvandijk mlvandijk Awaiting requested review from mlvandijk

Assignees

@olleolleolle olleolleolle

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@olleolleolle @plaindocs @mxygem