Skip to content

fix(exec-policy) No empty command lists#11397

Merged
dylan-hurd-oai merged 2 commits intomainfrom
dh--matched-rules-panic
Feb 11, 2026
Merged

fix(exec-policy) No empty command lists#11397
dylan-hurd-oai merged 2 commits intomainfrom
dh--matched-rules-panic

Conversation

@dylan-hurd-oai
Copy link
Collaborator

@dylan-hurd-oai dylan-hurd-oai commented Feb 11, 2026

Summary

This should rarely, if ever, happen in practice. But regardless, we should never provide an empty list of commands to ExecPolicy. This PR is almost entirely adding test around these cases.

Testing

  • Adds a bunch of unit tests for this

@dylan-hurd-oai dylan-hurd-oai force-pushed the dh--matched-rules-panic branch from b2bcc14 to a1c99bc Compare February 11, 2026 03:00
@dylan-hurd-oai dylan-hurd-oai enabled auto-merge (squash) February 11, 2026 03:05
@dylan-hurd-oai dylan-hurd-oai merged commit cc8c293 into main Feb 11, 2026
30 of 32 checks passed
@dylan-hurd-oai dylan-hurd-oai deleted the dh--matched-rules-panic branch February 11, 2026 03:22
@github-actions github-actions bot locked and limited conversation to collaborators Feb 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments