-
Notifications
You must be signed in to change notification settings - Fork 55
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
Depend on validator.pb.go from amphtml project. #88
Conversation
56a562a
to
9ece13d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, anything that touches transformer should be made in google3 (which is SoT)
This is covered by #92. |
9ece13d
to
6745351
Compare
This is a follow-up to bbfdb11 (ampproject#92) to pull the needed dependency into vendor/, and remove the unused file in pkg/. Also, add `prune.non-go = true` to Gopkg.toml, to eliminate a bunch of unnecessary files in vendor/.
6745351
to
2ab9e8d
Compare
PTAL - This is what's leftover from #92, and no longer covers |
Fixes #28. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete the generated proto from amppackager? Or was that handled in a separate commit?
That's in this commit; it just shows up as a rename. See https://github.com/ampproject/amppackager/pull/88/files#diff-94ee56b283d614195a3f7fd45652a572 or https://github.com/ampproject/amppackager/pull/88.patch. |
This removes the remaining file in pkg/. Also, add
prune.non-go = true
to Gopkg.toml, to eliminate a bunch of unnecessary files in vendor/.
Addresses #28.