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

tullia jobs: use generic gitrev for PRs #4844

Merged
merged 3 commits into from
Jan 26, 2023
Merged

Conversation

jbgi
Copy link
Contributor

@jbgi jbgi commented Jan 25, 2023

to avoid unnecessary builds of big artifact, like dockerImages.
Goal: bors ci build should not build anything if branch is up-to-date
and ci pass.

Some other minor clean-ups.

@jbgi jbgi force-pushed the skip-gitrev-for-pr branch from 9ce6a06 to c4035b9 Compare January 25, 2023 11:17
nix/tullia.nix Outdated Show resolved Hide resolved
flake.nix Outdated Show resolved Hide resolved
@jbgi jbgi force-pushed the skip-gitrev-for-pr branch from b1e519c to 14065eb Compare January 25, 2023 11:31
@jbgi jbgi requested a review from dermetfan January 25, 2023 11:37
 to avoid unnecessary builds of big artifact, like dockerImages.
 Goal: bors ci build should not build anything if branch is up-to-date
 and ci pass.
 Also bumped tullia.

 Some other minor clean-ups.
@jbgi jbgi force-pushed the skip-gitrev-for-pr branch from 14065eb to 0b5401e Compare January 26, 2023 09:55
@jbgi
Copy link
Contributor Author

jbgi commented Jan 26, 2023

bors merge

iohk-bors bot added a commit that referenced this pull request Jan 26, 2023
4844: tullia jobs: use generic gitrev for PRs r=jbgi a=jbgi

 to avoid unnecessary builds of big artifact, like dockerImages.
 Goal: bors ci build should not build anything if branch is up-to-date
 and ci pass.

 Some other minor clean-ups.

Co-authored-by: Jean-Baptiste Giraudeau <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 26, 2023

Build failed:

@jbgi jbgi force-pushed the skip-gitrev-for-pr branch from 0b5401e to 0ceeeec Compare January 26, 2023 10:38
@jbgi
Copy link
Contributor Author

jbgi commented Jan 26, 2023

bors merge

iohk-bors bot added a commit that referenced this pull request Jan 26, 2023
4844: tullia jobs: use generic gitrev for PRs r=jbgi a=jbgi

 to avoid unnecessary builds of big artifact, like dockerImages.
 Goal: bors ci build should not build anything if branch is up-to-date
 and ci pass.

 Some other minor clean-ups.

Co-authored-by: Jean-Baptiste Giraudeau <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 26, 2023

Build failed:

@jbgi
Copy link
Contributor Author

jbgi commented Jan 26, 2023

bors retry

iohk-bors bot added a commit that referenced this pull request Jan 26, 2023
4844: tullia jobs: use generic gitrev for PRs r=jbgi a=jbgi

 to avoid unnecessary builds of big artifact, like dockerImages.
 Goal: bors ci build should not build anything if branch is up-to-date
 and ci pass.

 Some other minor clean-ups.

Co-authored-by: Jean-Baptiste Giraudeau <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 26, 2023

Build failed:

@jbgi
Copy link
Contributor Author

jbgi commented Jan 26, 2023

bors retry

iohk-bors bot added a commit that referenced this pull request Jan 26, 2023
4844: tullia jobs: use generic gitrev for PRs r=jbgi a=jbgi

 to avoid unnecessary builds of big artifact, like dockerImages.
 Goal: bors ci build should not build anything if branch is up-to-date
 and ci pass.

 Some other minor clean-ups.

Co-authored-by: Jean-Baptiste Giraudeau <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 26, 2023

Build failed:

@jbgi
Copy link
Contributor Author

jbgi commented Jan 26, 2023

bors retry

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 26, 2023

Build succeeded:

@iohk-bors iohk-bors bot merged commit 0747007 into master Jan 26, 2023
@iohk-bors iohk-bors bot deleted the skip-gitrev-for-pr branch January 26, 2023 12:33
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