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: Retry nodeps build if it fails the first time #1283

Merged
merged 2 commits into from
Jun 14, 2022

Conversation

paulgessinger
Copy link
Member

For some reason the inclusion of ccache seems to have made the nodeps
build (specifically the boost build) fail sporadically. Since it works
most of the time and we want to keep using ccache for this build as
well, this PR attempts to just rertry the build step once if it fails.

For some reason the inclusion of ccache seems to have made the nodeps
build (specifically the boost build) fail sporadically. Since it works
most of the time and we want to keep using ccache for this build as
well, this PR attempts to just rertry the build step once if it fails.
@paulgessinger paulgessinger added this to the next milestone Jun 14, 2022
@paulgessinger
Copy link
Member Author

/cc @gagnonlg @andiwand

@andiwand andiwand self-requested a review June 14, 2022 12:56
Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

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

should we reference one of the failed workflows here?

@paulgessinger
Copy link
Member Author

@codecov
Copy link

codecov bot commented Jun 14, 2022

Codecov Report

Merging #1283 (1fc3256) into main (9920a81) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1283   +/-   ##
=======================================
  Coverage   47.48%   47.48%           
=======================================
  Files         376      376           
  Lines       19820    19820           
  Branches     9308     9308           
=======================================
  Hits         9412     9412           
  Misses       4018     4018           
  Partials     6390     6390           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@kodiakhq kodiakhq bot merged commit 93c608d into acts-project:main Jun 14, 2022
@gagnonlg gagnonlg mentioned this pull request Jun 14, 2022
@paulgessinger paulgessinger modified the milestones: next, v19.3.0 Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants