Skip to content

Don't let CI fail to check commits b/c git isn't configured in CI #734

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

Merged

Conversation

TheBlueMatt
Copy link
Collaborator

This fixes #733 by just setting a dummy git name/email before
calling git rebase in CI.

This PR is deliberately based on a stale version of git upstream to test it.

@TheBlueMatt TheBlueMatt force-pushed the 2020-10-ci-no-git-conf branch from 1fdd5ef to 867af52 Compare October 2, 2020 20:36
This fixes lightningdevkit#733 by just setting a dummy git name/email before
calling `git rebase` in CI.
@TheBlueMatt TheBlueMatt force-pushed the 2020-10-ci-no-git-conf branch from 867af52 to 89e40fe Compare October 2, 2020 20:37
@ariard
Copy link

ariard commented Oct 5, 2020

Code Review ACK 89e40fe, CI is triggered while the branch being based on old master branch.

@TheBlueMatt TheBlueMatt merged commit f35a5ce into lightningdevkit:main Oct 5, 2020
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.

Bug: Pre-commit check_commits failing due to empty ident name
2 participants