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

get_tools = tools #3988

Merged
merged 1 commit into from Sep 16, 2019
Merged

get_tools = tools #3988

merged 1 commit into from Sep 16, 2019

Conversation

tac0turtle
Copy link
Contributor

@tac0turtle tac0turtle commented Sep 16, 2019

  • in the makefile update get_tools was changed to just tools

ref #3949

Signed-off-by: Marko Baricevic marbar3778@yahoo.com

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- in the makefile update `get_tools` was changed to just `tools`

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
@tac0turtle tac0turtle added C:docs Component: Documentation ready-for-review labels Sep 16, 2019
@tac0turtle tac0turtle self-assigned this Sep 16, 2019
@@ -15,7 +15,7 @@ RUN apt-get update && apt-get install -y \

COPY Gopkg.toml /go/src/github.com/tendermint/abci/
COPY Gopkg.lock /go/src/github.com/tendermint/abci/
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this could use some updating 😆

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@21d46de). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3988   +/-   ##
=========================================
  Coverage          ?   66.92%           
=========================================
  Files             ?      219           
  Lines             ?    18485           
  Branches          ?        0           
=========================================
  Hits              ?    12372           
  Misses            ?     5190           
  Partials          ?      923

@tac0turtle tac0turtle merged commit 522a849 into master Sep 16, 2019
@tac0turtle tac0turtle deleted the marko/docsupdate branch September 16, 2019 08:46
@ebuchman
Copy link
Contributor

Can we link the original source for this change in the opening comment please?

cc @greg-szabo @mircea-c for awareness though they may already be aware

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:docs Component: Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants