Closed kengdoj closed 6 months ago
Thank you @dd8 for catching that.
I've opened https://github.com/act-rules/act-rules.github.io/pull/2144 to update the matching characters definition.
Call for Review ends on December 21st.
Removing from Call for Review as two reviewers were from the same organisation…
Added a comment regarding Pass 5. It feels to me like this scenario should be considered inapplicable, but that would require a modification to the applicability. The rule is consistent as-is, but it seems more accurate to say that the non-text content scenario is not applicable to this rule rather than to say that it passes it...
Update examples to include https://github.com/act-rules/act-rules.github.io/issues/1458#issuecomment-1555077770 Closes issue(s):
Need for Call for Review:
This will require a 2 weeks Call for Review << new rule, or substantial changes affecting a large number of test cases, if in doubt, use this. >>
Pull Request Etiquette
When creating PR:
develop
branch (left side).After creating PR:
Rule
,Definition
orChore
.When merging a PR:
How to Review And Approve