-
Notifications
You must be signed in to change notification settings - Fork 4
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
Remove bootstrap override hooks #1512
Conversation
Codecov Report
@@ Coverage Diff @@
## next #1512 +/- ##
==========================================
- Coverage 91.91% 91.91% -0.01%
==========================================
Files 277 277
Lines 4180 4179 -1
Branches 755 755
==========================================
- Hits 3842 3841 -1
Misses 313 313
Partials 25 25
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
size-limit report 📦
|
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.
LGTM
Thanks! |
@ahuth Ah, i wish i had seen this sooner; I was hoping to keep any breaking changes out of the release this week to exercise some PR sorting. Luckily this one is breaking as a technicality (unless someone is sneakily using those attributes...) |
Got excited. Can revert, @booc0mtaco. Although definitely no-one using them in GST. |
@ahuth It's all good; we can keep this in, and go to the next major. I wonder if the tooling knows how to handle a revert of a breaking change? |
EDS-850
Remove the
data-bootstrap-override
hooks. They're not needed anymore, because LP is staying on EDS v8.