Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Minor documentation fixes #148

Merged
merged 2 commits into from Jun 28, 2018
Merged

Minor documentation fixes #148

merged 2 commits into from Jun 28, 2018

Conversation

rickkw
Copy link
Contributor

@rickkw rickkw commented Jun 27, 2018

Description of the Change

Fixed a few documentation errors

@rickkw rickkw requested review from corindwyer and sargun June 27, 2018 22:13
README.md Outdated
1. gometalinter --install
1. go get -u github.com/kardianos/govendor

You would also need to make sure that your build environment (i.e. VM) has the following commands prior to building
Copy link
Contributor

Choose a reason for hiding this comment

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

delete would

README.md Outdated
gometalinter --install
go get -u github.com/kardianos/govendor
1. go get -u github.com/alecthomas/gometalinter
1. gometalinter --install
Copy link
Contributor

Choose a reason for hiding this comment

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

Aren't numbered lists in markdown the actual numbers such as:

1.
2.
3.

?

Copy link
Contributor

Choose a reason for hiding this comment

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

No.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1485

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 23.002%

Totals Coverage Status
Change from base Build 1475: 0.0%
Covered Lines: 2089
Relevant Lines: 9082

💛 - Coveralls

@codecov
Copy link

codecov bot commented Jun 28, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #148   +/-   ##
=======================================
  Coverage   33.84%   33.84%           
=======================================
  Files          61       61           
  Lines        7167     7167           
=======================================
  Hits         2426     2426           
  Misses       4443     4443           
  Partials      298      298

@sargun
Copy link
Contributor

sargun commented Jun 28, 2018

@rickkw feel free to hit the merge button when ready.

@rickkw rickkw merged commit 95dc5f3 into master Jun 28, 2018
@rickkw rickkw deleted the minor_doc_fixes branch July 12, 2018 18:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants