Skip to content

Microfix: Missing ',' in CLUTTER_RULES prevented rules to be lower priotized (as their two name were joined together)#3906

Merged
unp1 merged 1 commit into
mainfrom
fixClutterRuleList
Jul 10, 2026
Merged

Microfix: Missing ',' in CLUTTER_RULES prevented rules to be lower priotized (as their two name were joined together)#3906
unp1 merged 1 commit into
mainfrom
fixClutterRuleList

Conversation

@unp1

@unp1 unp1 commented Jul 10, 2026

Copy link
Copy Markdown
Member

Intended Change

CLUTTER_RULES defines a list of rules that should be given less priority when showing the taclet menu popup dialog. A missing comma prevented that for the two rules "apply_eq_monomials" and "eqTermCut".

Type of pull request

  • Bug fix (non-breaking change which fixes an issue)

Ensuring quality

  • I have tested the feature as follows: manual GUI tests

Additional information and contact(s)

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

@unp1 unp1 self-assigned this Jul 10, 2026
@unp1 unp1 added this to the v3.0.0 milestone Jul 10, 2026
@unp1 unp1 changed the title Missing ',' in CLUTTER_RULES prevented to rules to be hidden (as their two name were joined together) Microfix: Missing ',' in CLUTTER_RULES prevented to rules to be hidden (as their two name were joined together) Jul 10, 2026
@wadoon wadoon self-requested a review July 10, 2026 11:55
@unp1 unp1 changed the title Microfix: Missing ',' in CLUTTER_RULES prevented to rules to be hidden (as their two name were joined together) Microfix: Missing ',' in CLUTTER_RULES prevented rules to be lower priotized (as their two name were joined together) Jul 10, 2026
@unp1 unp1 added this pull request to the merge queue Jul 10, 2026
Merged via the queue into main with commit 08cd70a Jul 10, 2026
36 checks passed
@unp1 unp1 deleted the fixClutterRuleList branch July 10, 2026 12:42
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