Code monitors: add --glob and --exclude to git rev-parse allowlist

Review changes
Download
Patches
Plain diff

Merged Code monitors: add --glob and --exclude to git rev-parse allowlist

Overview 1
Commits 2
Pipelines 0
Changes 2

Merged Warren Gifford requested to merge cc/fix-glob-refs into main 2 years ago

Overview 1
Commits 2
Pipelines 0
Changes 2

Created by: camdencheek

This adds --glob and --exclude to the allowlist for git rev-parse arguments. Their absence was causing failures when we'd attempt to create a code monitor that tracks a globbed set of branches.

Fixes #https://github.com/sourcegraph/sourcegraph/issues/36661

Test plan

Manually tested that creating a monitor locally works now and does not error.

Merge request reports

Activity

Filter activity

Approvals
Assignees & reviewers
Comments (from bots)
Comments (from users)
Commits & branches
Edits
Labels
Lock status
Mentions
Merge request status
Tracking

Please register or sign in to reply

0 Assignees

None

Select assignees

0 Reviewers

None

Request review from

Labels

cla-signed team/search-product

Select labels

Manage project labels

Milestone

None

Time tracking

No estimate or time spent

1 Participant

There are currently no pipelines.

To run a merge request pipeline, the jobs in the CI/CD configuration file must be configured to run in merge request pipelines and you must have sufficient permissions in the source project.