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

Run dep check in travis-ci #332

Closed
wants to merge 2 commits into from
Closed

Conversation

mestery
Copy link
Contributor

@mestery mestery commented Aug 21, 2018

This is from a discussion on #327

Similar to the vpp-agent [1] [2], run dep check in travis-ci with each
commit and fail if someone adds vendored code without proper vendoring.

[1] https://github.com/ligato/vpp-agent/blob/pantheon-dev/.travis.yml#L41
[2] https://github.com/ligato/vpp-agent/blob/pantheon-dev/Makefile#L179-L181

Signed-off-by: Kyle Mestery [email protected]

This is from a discussion on ligato#327

Similar to the vpp-agent [1] [2], run `dep check` in travis-ci with each
commit and fail if someone adds vendored code without proper vendoring.

[1] https://github.com/ligato/vpp-agent/blob/pantheon-dev/.travis.yml#L41
[2] https://github.com/ligato/vpp-agent/blob/pantheon-dev/Makefile#L179-L181

Signed-off-by: Kyle Mestery <[email protected]>
@mestery
Copy link
Contributor Author

mestery commented Aug 21, 2018

@ondrej-fabry Did we want to iterate on this one or #331?

@ondrej-fabry
Copy link
Member

@mestery
Copy link
Contributor Author

mestery commented Aug 21, 2018

Closing in lieu of #331.

@mestery mestery closed this Aug 21, 2018
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.

2 participants