Skip to content

GRSlime rules inherit from the RB rules thus are incorrectly identified#202

Merged
marschall merged 1 commit into
SeasideSt:masterfrom
tukanos:uniqueIdentifierName
Jul 8, 2026
Merged

GRSlime rules inherit from the RB rules thus are incorrectly identified#202
marschall merged 1 commit into
SeasideSt:masterfrom
tukanos:uniqueIdentifierName

Conversation

@tukanos

@tukanos tukanos commented Apr 27, 2026

Copy link
Copy Markdown

Pharo uses unique identifier to identify rule class. Since GRSlime rules inherit from RB rules they inherit also their unique identifier. This patch creates unique identifier for the GRSlime classes.

For more information see test ReSmallLintTest>>testUniqueIdentifierName in Pharo

@tukanos
tukanos force-pushed the uniqueIdentifierName branch from 31149c7 to 10eb5ef Compare April 27, 2026 09:14
Pharo uses unique identifier to identify rule class.  Since GRSlime
rules inherit from RB rules they inherit also their unique identifier.
This patch creates unique identifier for the GRSlime classes.

For more information see test ReSmallLintTest>>testUniqueIdentifierName
in Pharo
@tukanos
tukanos force-pushed the uniqueIdentifierName branch from 10eb5ef to 0689aef Compare April 27, 2026 10:03
@marschall
marschall self-requested a review July 8, 2026 15:08

@marschall marschall 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.

Thank you

@marschall
marschall merged commit a349135 into SeasideSt:master Jul 8, 2026
@tukanos
tukanos deleted the uniqueIdentifierName branch July 14, 2026 19:03
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