Skip to content

Add go.mod and go.sum to lock dependencies#830

Closed
Iceyer wants to merge 1 commit into
opencontainers:mainfrom
Iceyer:master
Closed

Add go.mod and go.sum to lock dependencies#830
Iceyer wants to merge 1 commit into
opencontainers:mainfrom
Iceyer:master

Conversation

@Iceyer
Copy link
Copy Markdown

@Iceyer Iceyer commented Mar 25, 2021

No description provided.

@Iceyer Iceyer changed the title Add go.mod and go.sum to lock dependencies WIP: Add go.mod and go.sum to lock dependencies Mar 25, 2021
@Iceyer Iceyer changed the title WIP: Add go.mod and go.sum to lock dependencies Add go.mod and go.sum to lock dependencies Mar 25, 2021
@Iceyer
Copy link
Copy Markdown
Author

Iceyer commented Mar 25, 2021

$ docker pull vbatts/pandoc

Using default tag: latest

Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

The command "docker pull vbatts/pandoc" failed and exited with 1 during .

could anyone help restart the ci build job?

@brendandburns @cyphar @jbouzane

@fuweid
Copy link
Copy Markdown
Member

fuweid commented Mar 25, 2021

@Iceyer #814 will fix the issue. After the pr merged, you can rebase/repush to restart CI.

@fuweid
Copy link
Copy Markdown
Member

fuweid commented Mar 31, 2021

the CI issue has been fixed. Needs rebase here

Copy link
Copy Markdown
Member

@vbatts vbatts left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread go.mod Outdated
gometalinter is loss of maintenance, and could not build with go mod.
So add GO111MODULE=off when use gometalinter.

See also: alecthomas/gometalinter#590.

Signed-off-by: Iceyer <me@iceyer.net>
@vbatts
Copy link
Copy Markdown
Member

vbatts commented Jul 9, 2021

didn't even realize this is a duplicate of #810, which was opened before this one.

@vbatts vbatts closed this Jul 9, 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