Skip to content

Conversation

@edg2s
Copy link
Member

@edg2s edg2s commented Oct 8, 2025

$x.toggleClass(false) which turns off all classes was deprecated
in 3.0, and will be removed in 4. Other signatures are unaffected,
so make this a separate rule to no-class.

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (2ff8458) to head (9d9786a).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #352   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          105       106    +1     
  Lines          635       651   +16     
=========================================
+ Hits           635       651   +16     
Files with missing lines Coverage Δ
src/index.js 100.00% <ø> (ø)
src/rules/no-toggle-all-classes.js 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@edg2s
Copy link
Member Author

edg2s commented Oct 8, 2025

-1 - should merge with no-class

@edg2s edg2s changed the base branch from master to deps October 9, 2025 08:41
@edg2s edg2s changed the base branch from deps to master October 9, 2025 08:41
@edg2s edg2s changed the title New rule: no-toggle-class; included in deprecated-3.0 with option .toggleClass(boolean|undefined) is deprecated. Oct 9, 2025
$x.toggleClass(false) which turns off all classes was deprecated
in 3.0, and will be removed in 4. Other signatures are unaffected,
so make this a separate rule to `no-class`.
@jdforrester
Copy link
Member

The git commit title should be "New rule: no-toggle-all-classes" etc.; as-is, this is an Issue title.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants