Skip to content

github: Fix code scanning alerts - #305

Merged
bkeryan merged 3 commits into
mainfrom
users/bkeryan/fix-zizmor-errors
Nov 3, 2026
Merged

bkeryan merged 3 commits into
mainfrom
users/bkeryan/fix-zizmor-errors

Conversation

@bkeryan

@bkeryan bkeryan commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

What does this Pull Request accomplish?

Fix zizmor errors:

  • Use new self-repository syntax ($/...)
  • Ignore dangerous-triggers error for sync_github_issues_to_azdo.yml. This usage of issue_comment does not checkout the repo or run any code other than an action which is designed to be used with the issues and issue_comment triggers.

Update gh-action-pypi-publish to v1.14.3.

Why should this Pull Request be merged?

Fix code scanning alerts.

What testing has been done?

PR build

@bkeryan
bkeryan requested a review from csjall as a code owner November 3, 2026 01:12
@bkeryan
bkeryan requested a review from mjohanse-emr November 3, 2026 01:13
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Test Results

    50 files  +    2      50 suites  +2   17m 53s ⏱️ + 1m 9s
 2 313 tests ±    0   2 313 ✅ ±    0  0 💤 ±0  0 ❌ ±0 
59 628 runs  +2 300  59 628 ✅ +2 300  0 💤 ±0  0 ❌ ±0 

Results for commit a3ae0a8. ± Comparison against base commit 8ef370f.

@bkeryan
bkeryan merged commit deae618 into main Nov 3, 2026
39 checks passed
@bkeryan
bkeryan deleted the users/bkeryan/fix-zizmor-errors branch November 3, 2026 17:34
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