Skip to content

Comments

feat: restrict-plus-operands: improve diagnostic#725

Merged
graphite-app[bot] merged 1 commit intomainfrom
02-21-fea_restrict-plus-operands_improve_diagnostic
Feb 22, 2026
Merged

feat: restrict-plus-operands: improve diagnostic#725
graphite-app[bot] merged 1 commit intomainfrom
02-21-fea_restrict-plus-operands_improve_diagnostic

Conversation

@camchenry
Copy link
Member

@camchenry camchenry commented Feb 22, 2026

NOTE: There is an issue with help messages not showing up in snapshots, I will fix this in the following PR.

This rule's error message is packed with both type information and help information (what types are allowed). I've moved the resolved type info to be labeled ranges, and moved the message about what types are allowed to the help message. This should keep the information a little bit more structured and not crowd the main error message as much.

Copy link
Member Author

camchenry commented Feb 22, 2026


How to use the Graphite Merge Queue

Add the label 0-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@camchenry camchenry changed the title fea: restrict-plus-operands: improve diagnostic feat: restrict-plus-operands: improve diagnostic Feb 22, 2026
@camchenry camchenry marked this pull request as ready for review February 22, 2026 04:33
@graphite-app graphite-app bot changed the base branch from 02-21-feat_restrict-template-expressions_move_type_to_labeled_range to graphite-base/725 February 22, 2026 13:47
@camc314 camc314 self-assigned this Feb 22, 2026
Copy link
Contributor

camc314 commented Feb 22, 2026

Merge activity

@graphite-app graphite-app bot changed the base branch from graphite-base/725 to main February 22, 2026 13:53
NOTE: There is an issue with help messages not showing up in snapshots, I will fix this in the following PR.

This rule's error message is packed with both type information and help information (what types are allowed). I've moved the resolved type info to be labeled ranges, and moved the message about what types are allowed to the help message. This should keep the information a little bit more structured and not crowd the main error message as much.
@graphite-app graphite-app bot force-pushed the 02-21-fea_restrict-plus-operands_improve_diagnostic branch from ed87449 to baedad5 Compare February 22, 2026 13:54
@graphite-app graphite-app bot merged commit baedad5 into main Feb 22, 2026
8 checks passed
@graphite-app graphite-app bot deleted the 02-21-fea_restrict-plus-operands_improve_diagnostic branch February 22, 2026 14:01
@graphite-app graphite-app bot removed the 0-merge label Feb 22, 2026
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