-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chore merge main from 52cc934 to 4e35f64 #2474
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove from our files references to // Boosted mod: no $accordion-button-focus-border-color
and // Boosted mod: no --#{$prefix}accordion-btn-focus-border-color
twbs/bootstrap@52cc934: the bump of copyright year to 2024 was already done in Boostedjs-lint
wasn't checking JS files 🙃 I've fixed the files in error and excluded/stories/create-stories-from-doc.js
that contained too much errors linked to await and stuff; too risky to fix)twbs/bootstrap@e4d8f14: nothing to do here since we don't userelease-drafter
twbs/bootstrap@409fd23: we don't use:rtl:options
to handle carousel prev/next icons so I don't think there's anything to do here.twbs/bootstrap@4e35f64: nothing to do here since we don't use--#{$prefix}accordion-btn-focus-border-color
nor$accordion-button-focus-border-color