Skip to content

[hooks_runner] Add PATHEXT to environment variables allowlist - #3539

Merged
auto-submit[bot] merged 1 commit into
mainfrom
hooks-runner-pathext
Aug 11, 2026
Merged

auto-submit[bot] merged 1 commit into
mainfrom
hooks-runner-pathext

Conversation

@dcharkes

Copy link
Copy Markdown
Collaborator

When using package:process it tries to look at PATHEXT before doing process invocations. And we're trying to start to use that in package:native_toolchain_c.

Context:

@goderbauer goderbauer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-actions

github-actions Bot commented Aug 11, 2026 •

Copy link
Copy Markdown

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

API leaks ✔️

The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.

Package Leaked API symbol Leaking sources

This check can be disabled by tagging the PR with skip-leaking-check.

Breaking changes ✔️
Package Change Current Version New Version Needed Version Looking good?
hooks_runner Non-Breaking 1.6.1 1.6.2-wip 1.6.2-wip ✔️

This check can be disabled by tagging the PR with skip-breaking-check.

@auto-submit
auto-submit Bot merged commit 263decb into main Aug 11, 2026
36 checks passed
@auto-submit
auto-submit Bot deleted the hooks-runner-pathext branch August 11, 2026 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants