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.

Sign up for GitHub

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

Jump to bottom

Retire vendoring #1502

Merged
merged 1 commit into from
Apr 6, 2021
Merged

Retire vendoring #1502

merged 1 commit into from
Apr 6, 2021

Conversation

lucacome
Copy link
Member

@lucacome lucacome commented Apr 2, 2021

  • Removed vendor folder and -mod=vendor from build
  • Modified codegen script to use go cache
  • Added codegen check to workflows

@lucacome lucacome requested a review from a team April 2, 2021 23:52
@lucacome lucacome self-assigned this Apr 2, 2021
@lucacome lucacome requested review from ciarams87 and soneillf5 and removed request for a team April 2, 2021 23:52
@github-actions github-actions bot added chore Pull requests for routine tasks dependencies Pull requests that update a dependency file documentation Pull requests/issues for documentation labels Apr 2, 2021
@lucacome lucacome requested a review from pleshakov April 5, 2021 15:49
@lucacome lucacome enabled auto-merge (squash) April 5, 2021 19:52
@lucacome lucacome merged commit 6b2254f into master Apr 6, 2021
@lucacome lucacome deleted the chore/remove-vendor-folder branch April 9, 2021 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@soneillf5 soneillf5 soneillf5 approved these changes

@pleshakov pleshakov pleshakov approved these changes

@ciarams87 ciarams87 Awaiting requested review from ciarams87

Assignees

@lucacome lucacome

Labels
chore Pull requests for routine tasks dependencies Pull requests that update a dependency file documentation Pull requests/issues for documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lucacome @pleshakov @soneillf5