Skip to content

Cfg: Distinguish parameters from their patterns.#21999

Merged
aschackmull merged 2 commits into
github:mainfrom
aschackmull:cfg/parameter-pattern
Jun 18, 2026
Merged

Cfg: Distinguish parameters from their patterns.#21999
aschackmull merged 2 commits into
github:mainfrom
aschackmull:cfg/parameter-pattern

Conversation

@aschackmull

Copy link
Copy Markdown
Contributor

This moves the AST signature for parameters in the CFG lib closer to the unified AST and fits more closely with Ruby.

@aschackmull aschackmull requested review from a team as code owners June 17, 2026 16:22
Copilot AI review requested due to automatic review settings June 17, 2026 16:22
@aschackmull aschackmull requested a review from a team as a code owner June 17, 2026 16:22

Copilot AI 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.

Copilot encountered an error: Your billing is not configured or you have Copilot licenses from multiple standalone organizations or enterprises. To use premium requests, select a billing entity via the GitHub site, under Settings > Copilot > Features.

@hvitved

hvitved commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

@aschackmull

aschackmull commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author
Rerun has been triggered: 1 restarted 🚀

@aschackmull aschackmull added the no-change-note-required This PR does not need a change note label Jun 18, 2026
@aschackmull aschackmull force-pushed the cfg/parameter-pattern branch from 75f31f6 to f844cd3 Compare June 18, 2026 09:00
@aschackmull

aschackmull commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author
Rerun has been triggered: 3 restarted 🚀

@aschackmull aschackmull merged commit 779309e into github:main Jun 18, 2026
140 checks passed
@aschackmull aschackmull deleted the cfg/parameter-pattern branch June 18, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C# Java no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants