Skip to content

Add quotes to status value for results delivery#147

Merged
geoffg-sentry merged 1 commit into
mainfrom
origin/geoffg/fix-secretscanning-json
Oct 30, 2024
Merged

Add quotes to status value for results delivery#147
geoffg-sentry merged 1 commit into
mainfrom
origin/geoffg/fix-secretscanning-json

Conversation

@geoffg-sentry
Copy link
Copy Markdown
Contributor

Whoops, I missed this in my review of #146. Value of status needs to be quoted.

Currently the log looks like:

{"event":"github_secret_scanning", "status":success, "createdAt":"2024-10-30T17:11:19Z", "repo":"getsentry/sentry","pull_request":"https://github.com/getsentry/sentry/pull/79995","actor":"some.user"}

And now it'll be

{"event":"github_secret_scanning", "status":"success", "createdAt":"2024-10-30T17:11:19Z", "repo":"getsentry/sentry","pull_request":"https://github.com/getsentry/sentry/pull/79995","actor":"some.user"}

Whoops, I missed this in my review of #146. Value of status needs to be quoted.

Currently the log looks like
{"event":"github_secret_scanning", "status":success, "createdAt":"2024-10-30T17:20:07Z", "repo":"getsentry/gocd-jsonnet","pull_request":"https://github.com/getsentry/gocd-jsonnet/pull/83","actor":"IanWoodard"}
@geoffg-sentry geoffg-sentry enabled auto-merge (squash) October 30, 2024 18:24
@geoffg-sentry geoffg-sentry merged commit f0fdb2d into main Oct 30, 2024
@geoffg-sentry geoffg-sentry deleted the origin/geoffg/fix-secretscanning-json branch October 30, 2024 18:30
Copy link
Copy Markdown

@jeff-a-holland-codecov jeff-a-holland-codecov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

3 participants