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

Go module conversion #346

Merged
merged 2 commits into from
Sep 16, 2019
Merged

Go module conversion #346

merged 2 commits into from
Sep 16, 2019

Conversation

banaag
Copy link
Collaborator

@banaag banaag commented Sep 13, 2019

Go dep to go-module conversion. Addresses #270

@banaag banaag requested a review from twifkak September 13, 2019 16:36
@banaag banaag force-pushed the go-module-conversion branch 3 times, most recently from 2e9d7a1 to aa58cb7 Compare September 16, 2019 16:51
@banaag banaag force-pushed the go-module-conversion branch from aa58cb7 to e1f7b92 Compare September 16, 2019 16:55
Copy link
Member

@twifkak twifkak left a comment

Choose a reason for hiding this comment

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

Thanks! Couple more things.

  • git rm Gopkg.{toml,lock}.
  • Add an empty cmd/gateway_server/go.mod. (This is the equivalent of the "ignored" line in the old Gopkg.toml.) Rerun go mod tidy. This should reduce the diff a lot.

Copy link
Member

@twifkak twifkak left a comment

Choose a reason for hiding this comment

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

@banaag banaag merged commit 83729a6 into ampproject:master Sep 16, 2019
twifkak added a commit to twifkak/amppackager that referenced this pull request Dec 5, 2019
where available. This reflects the change from dep to go1.11 modules in
83729a6 (ampproject#346).
twifkak added a commit that referenced this pull request Dec 9, 2019
where available. This reflects the change from dep to go1.11 modules in
83729a6 (#346).
@banaag banaag added the acme All ACME related PRs label Feb 24, 2020
@banaag banaag deleted the go-module-conversion branch March 5, 2020 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
acme All ACME related PRs cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants