development/github-desktop
development/github-desktop
1
0
Fork 0
mirror of https://github.com/desktop/desktop synced 2024-10-31 05:19:03 +00:00
Code Issues Projects Releases Packages Wiki Activity

Don't assign gpgSignEnabled more than once

Browse source
This commit is contained in:
Markus Olsson 2023-08-31 09:46:43 +02:00
parent 5d2cd614c9
commit 541b2659bd
1 changed files with 6 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
app/src/lib/helpers/repo-rules.ts
View file

@ -71,7 +71,7 @@ export async function parseRepoRules(
repository: Repository
): Promise<RepoRulesInfo> {
const info = new RepoRulesInfo()
let gitConfigKeySet: boolean | null = null
let gpgSignEnabled: boolean | undefined = undefined
for (const rule of rules) {
// if a ruleset is null/undefined, then act as if the rule doesn't exist because
@ -102,17 +102,12 @@ export async function parseRepoRules(
break
case APIRepoRuleType.RequiredSignatures:
// check if the user has commit signing configured. if they do, the rule passes
// and doesn't need to be warned about.
if (gitConfigKeySet === null) {
gitConfigKeySet = await getBooleanConfigValue(
repository,
'commit.gpgsign',
false
)
}
// check if the user has commit signing configured. if they do, the rule
// passes and doesn't need to be warned about.
gpgSignEnabled ??=
(await getBooleanConfigValue(repository, 'commit.gpgsign')) ?? false
if (gitConfigKeySet !== true) {
if (gpgSignEnabled !== true) {
info.signedCommitsRequired =
info.signedCommitsRequired !== true ? enforced : true
}