Skip to content

[TASK] Rename Rule::$rule to Rule::$propertyName#1506

Merged
oliverklee merged 2 commits into
mainfrom
task/rename-rule-rule
Feb 12, 2026
Merged

[TASK] Rename Rule::$rule to Rule::$propertyName#1506
oliverklee merged 2 commits into
mainfrom
task/rename-rule-rule

Conversation

@JakeQZ

@JakeQZ JakeQZ commented Feb 12, 2026

Copy link
Copy Markdown
Collaborator

This has become a misnomer. Rules are the outer enclosing blocks.

Deprecate the existing setters and getters and add new ones.

(The name of the Rule class itself is also a misnomer.)

Ref: https://developer.mozilla.org/en-US/docs/Web/API/CSS_Object_Model/CSS_Declaration

@JakeQZ JakeQZ requested a review from oliverklee February 12, 2026 02:45
@JakeQZ JakeQZ self-assigned this Feb 12, 2026
@JakeQZ JakeQZ added cleanup deprecation A method, property, or some functionality has been deprecated. refactor For PRs that refactor code without changing functionality labels Feb 12, 2026
@coveralls

coveralls commented Feb 12, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 72.813% (-0.2%) from 72.984%
when pulling 87993ea on task/rename-rule-rule
into 252bbe1 on main.

This has become a misnomer.  Rules are the outer enclosing blocks.

Deprecate the existing setters and getters and add new ones.

(The name of the `Rule` class itself is also a misnomer.)

Ref: https://developer.mozilla.org/en-US/docs/Web/API/CSS_Object_Model/CSS_Declaration
@JakeQZ JakeQZ force-pushed the task/rename-rule-rule branch from 9387189 to ebbb247 Compare February 12, 2026 10:18
Comment thread src/Rule/Rule.php Outdated
@JakeQZ JakeQZ requested a review from oliverklee February 12, 2026 12:23
@oliverklee oliverklee merged commit 1ce3447 into main Feb 12, 2026
24 checks passed
@oliverklee oliverklee deleted the task/rename-rule-rule branch February 12, 2026 12:39
JakeQZ added a commit that referenced this pull request Feb 13, 2026
This replaces `Rule\Rule`, following #1508.

Also use `getPropertyName()` rather than `getRule()` which was deprecated
in #1506.

This covers both the 'functional' and 'unit' flavours of the `TestCase`, and
the trait that does most of the heavy lifting for the 'unit' flavour.
JakeQZ added a commit that referenced this pull request Feb 13, 2026
This method replaces `getRule()`, which is deprecated as of #1506.
JakeQZ added a commit that referenced this pull request Feb 13, 2026
This method replaces `getRule()`, which is deprecated as of #1506.
oliverklee pushed a commit that referenced this pull request Feb 14, 2026
This method replaces `getRule()`, which is deprecated as of #1506.
JakeQZ added a commit that referenced this pull request Feb 14, 2026
This method replaces `getRule()`, which is deprecated as of #1506.
oliverklee pushed a commit that referenced this pull request Feb 14, 2026
This method replaces `getRule()`, which is deprecated as of #1506.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup deprecation A method, property, or some functionality has been deprecated. refactor For PRs that refactor code without changing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants