Skip to content

Add go.mod for Go modules - #1187

Merged
vishr merged 3 commits into
labstack:masterfrom
KimMachineGun:add-gomod
Sep 4, 2018
Merged

Add go.mod for Go modules#1187
vishr merged 3 commits into
labstack:masterfrom
KimMachineGun:add-gomod

Conversation

@KimMachineGun

Copy link
Copy Markdown
Contributor

No description provided.

@KimMachineGun
KimMachineGun deleted the add-gomod branch August 29, 2018 04:40
@KimMachineGun
KimMachineGun restored the add-gomod branch August 30, 2018 01:33
@KimMachineGun KimMachineGun reopened this Aug 30, 2018
@codecov

codecov Bot commented Aug 30, 2018

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1187   +/-   ##
=======================================
  Coverage   81.34%   81.34%           
=======================================
  Files          25       25           
  Lines        1919     1919           
=======================================
  Hits         1561     1561           
  Misses        250      250           
  Partials      108      108

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 9082231...9b4d027. Read the comment docs.

@KimMachineGun

Copy link
Copy Markdown
Contributor Author

Sync with labstack's master branch.

@vishr

vishr commented Aug 30, 2018

Copy link
Copy Markdown
Member

@im-kulikov Can you review this?

@im-kulikov

im-kulikov commented Aug 31, 2018

Copy link
Copy Markdown
Contributor

@vishr yep..
github.com/labstack/gommon same issue (#1185)

After version v0.2.1 format was changed...

  • before vX.Y.Z
  • after X.Y.Z

Comment thread echo.go Outdated
Learn more at https://echo.labstack.com
*/
package echo
package echo // import "github.com/labstack/echo"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not required

Comment thread go.mod

@im-kulikov im-kulikov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

pls add go.sum

package echo // import "github.com/labstack/echo"

not required

@KimMachineGun

Copy link
Copy Markdown
Contributor Author

Thank you for reviewing my PR. I will modify that and recommit it ASAP.

@KimMachineGun

Copy link
Copy Markdown
Contributor Author

I resolved it.

@im-kulikov

Copy link
Copy Markdown
Contributor

LGTM

@vishr
vishr merged commit 54d5613 into labstack:master Sep 4, 2018
@KimMachineGun
KimMachineGun deleted the add-gomod branch October 14, 2018 11:09
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