Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

batches: make repo exclusion exact #55096

Draft
burmudar wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

batches: make repo exclusion exact #55096

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion client/web/src/enterprise/batches/batch-spec/yaml-util.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ const appendExcludeRepoToQuery = (spec: string, ast: YAMLMap, repo: string): YAM
const isQuoted = queryValue.doubleQuoted || queryValue.singleQuoted || false

// If the value is not quoted, we need to escape characters
const excludeQualifierString = isQuoted ? ` -repo:${repo}` : ` -repo:${escapeRegExp(repo)}`
const excludeQualifierString = isQuoted ? ` -repo:^${repo}$` : ` -repo:^${escapeRegExp(repo)}$`
// If the value is quoted, we also need to shift the slice position so that the string
// is inserted inside of the quotes
const slicePosition = isQuoted ? queryValue.endPosition - 1 : queryValue.endPosition
Expand Down