Skip to content

chore: do not sign commits in tests#311

Merged
nlf merged 1 commit intomainfrom
nlf/tweak-tests
May 3, 2023
Merged

chore: do not sign commits in tests#311
nlf merged 1 commit intomainfrom
nlf/tweak-tests

Conversation

@nlf
Copy link
Copy Markdown
Contributor

@nlf nlf commented May 3, 2023

currently running the tests creates git repos and commits using your default git configuration. for me personally, this means that i get a popup confirming the signing request for these commits since i have my git configured to always sign. this means when running tests, i get a lot of notifications. by forcibly telling git to not sign these commits, this no longer happens, and since signatures are not important for test fixtures that are being discarded anyway felt like a quick usability win here.

@nlf nlf requested a review from a team as a code owner May 3, 2023 17:06
@nlf nlf requested a review from lukekarrys May 3, 2023 17:06
@nlf nlf merged commit 25ee5b6 into main May 3, 2023
@nlf nlf deleted the nlf/tweak-tests branch May 3, 2023 17:15
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