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

Add check for docs #138

Merged
merged 7 commits into from
Apr 17, 2018
Merged

Add check for docs #138

merged 7 commits into from
Apr 17, 2018

Conversation

domgreen
Copy link
Contributor

This commit adds a check for any change to the documentation that has not been correctly committed. It does this by regenerating the docs in Travis and then seeing if there are any changes due for staging in git.

Have also added a CONTRIBUTING.md file so that it is easier for new contributors to understand the flow.

@domgreen domgreen self-assigned this Apr 17, 2018
@domgreen domgreen requested review from yifanzz and mwitkow April 17, 2018 10:20
@codecov-io
Copy link

codecov-io commented Apr 17, 2018

Codecov Report

Merging #138 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #138   +/-   ##
=======================================
  Coverage   73.23%   73.23%           
=======================================
  Files          36       36           
  Lines        1300     1300           
=======================================
  Hits          952      952           
  Misses        300      300           
  Partials       48       48

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c7eebe4...af290ac. Read the comment docs.

scripts/docs.sh Outdated
if [ "$count" = "0" ]; then
return 0
else
return 1
Copy link
Member

Choose a reason for hiding this comment

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

Can you make this echo useful information such as:
"Your markdown docs seem to be out of sync with the package docs. Please run make and consult CONTRIBUTING.MD"

.travis.yml Outdated
@@ -15,7 +15,8 @@ install:
- dep ensure

script:
- make test
- make test
- make checkdocs
Copy link
Member

Choose a reason for hiding this comment

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

Make sure the checkdocs are first.

@domgreen
Copy link
Contributor Author

Also need to add realpath

@domgreen
Copy link
Contributor Author

@mwitkow have made changes also note the annoying realpath change that I have added ... this is due to Travis not having realpath installed ... by the looks of things realpath is a can of worms.

@mwitkow
Copy link
Member

mwitkow commented Apr 17, 2018

Legit. You don't need real path though, there's a bash one liner that does that.

@domgreen
Copy link
Contributor Author

Updated as per discussion to change to bash one liner 👍 merging

@domgreen domgreen merged commit bc17532 into grpc-ecosystem:master Apr 17, 2018
@domgreen domgreen deleted the add_check_for_docs branch April 17, 2018 12:10
stanhu added a commit to stanhu/go-grpc-middleware that referenced this pull request Feb 13, 2021
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