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

ci: migrate long sims #6084

Merged
merged 64 commits into from
Apr 28, 2020
Merged

ci: migrate long sims #6084

merged 64 commits into from
Apr 28, 2020

Conversation

tac0turtle
Copy link
Member

@tac0turtle tac0turtle commented Apr 27, 2020

Description

cancel ci if no .go files have been touched. There is no clean way of doing this so i opted to exit 1 on the diff job if no go files were changed and have builds depend on diff passing

  • migrate proto checks to github actions

  • providing make commands to use buf within docker

  • test-sim-multi-seed-long migration to github actions. Follows same logic and only runs on release branches

  • add protobuf section to contributing.md

@codecov
Copy link

codecov bot commented Apr 27, 2020

Codecov Report

Merging #6084 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6084   +/-   ##
=======================================
  Coverage   54.57%   54.57%           
=======================================
  Files         436      436           
  Lines       26296    26296           
=======================================
  Hits        14352    14352           
  Misses      10952    10952           
  Partials      992      992           

@tac0turtle tac0turtle added the R4R label Apr 28, 2020
@tac0turtle tac0turtle removed the R4R label Apr 28, 2020
@tac0turtle
Copy link
Member Author

need to debug the proto-breakage ci

@tac0turtle tac0turtle added the R4R label Apr 28, 2020
@tac0turtle tac0turtle added the A:automerge Automatically merge PR once all prerequisites pass. label Apr 28, 2020
@alessio alessio merged commit 4b3eb0e into master Apr 28, 2020
@alessio alessio deleted the marko/githubactions branch April 28, 2020 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:automerge Automatically merge PR once all prerequisites pass. T: CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants