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

GithubCI: upgrade actions to latest version #228

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aarani
Copy link
Contributor

@aarani aarani commented Oct 12, 2023

Installing git before checkout was necessary to prevent this error when git wants to interact with the checked out repo (Probably because it's downloaded using
Github's REST API):

fatal: not a git repository (or any parent up to mount point /)

Installing git before checkout also removes the problem with using submodule parameter in checkout action so the comments we had are no longer applicable.

@knocte knocte force-pushed the master branch 2 times, most recently from 1a3be72 to 265b103 Compare October 16, 2023 05:05
@aarani aarani force-pushed the ciUpgrade branch 2 times, most recently from e174e79 to 456d104 Compare October 17, 2023 11:28
@knocte
Copy link
Member

knocte commented Oct 19, 2023

Are you sure DEBIAN_FRONTEND is needed when installing git?

@knocte
Copy link
Member

knocte commented Oct 19, 2023

You're missing the why in the commit msg.

Installing git before checkout was necessary to prevent this
error when git wants to interact with the checked out repo
(Probably because it's downloaded using Github's REST API):
```
fatal: not a git repository (or any parent up to mount point /)
```

Installing git before checkout also removes the problem with
using submodule parameter in checkout action so the comments we
had are no longer applicable.

This was done to fix errors we had with downloading actions [1].

[1] nblockchain#224 (comment)
@knocte
Copy link
Member

knocte commented Oct 21, 2023

The why should be the first thing in the commit message body, not the last thing.

@knocte
Copy link
Member

knocte commented Oct 21, 2023

Also, PR description and commit msg body should match, given that there's only 1 commit.

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