Skip to content

Add newer (v2.18) git to CI images#753

Merged
bam241 merged 4 commits into
svalinn:developfrom
gonuke:newer_git_ci
Jul 12, 2021
Merged

Add newer (v2.18) git to CI images#753
bam241 merged 4 commits into
svalinn:developfrom
gonuke:newer_git_ci

Conversation

@gonuke

@gonuke gonuke commented Jul 12, 2021

Copy link
Copy Markdown
Member

Description

Install newer git during creation of CI images.

Motivation and Context

Continuous integration relies on having a full peer of the git repo. This was standard during CI on CircleCI, but the Github Action to fetch the repo requires a newer version of git (v2.18+) to accomplish this.

Changes

This changes the packages installed when making the base CI images.

@gonuke

gonuke commented Jul 12, 2021

Copy link
Copy Markdown
Member Author

I think these still need to be tested manually, is that right?

@gonuke

gonuke commented Jul 12, 2021

Copy link
Copy Markdown
Member Author

These are being tested on my repo... not sure if this is intended, but maybe a good outcome???

@gonuke

gonuke commented Jul 12, 2021

Copy link
Copy Markdown
Member Author

It seems that the test fail on my repo because I don't have push permission to the GHCR (?)

@bam241

bam241 commented Jul 12, 2021

Copy link
Copy Markdown
Member

This looks good .

I also think that pushing is not working. (we should probably add branch push restriction to avoid random run like this...)

I'll issue a PR to have the build run, and maybe tested on PR

if that crashes after merge, I'll take full responsibility, and will work on a fix !

Thx @gonuke, Merging

@bam241 bam241 merged commit 4ec7623 into svalinn:develop Jul 12, 2021
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