-
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) patched commit → a1a9895 #1751
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
a440ad5
to
6958ea6
Compare
So I'm gonna tackle twbs/bootstrap@7cb376a + twbs/bootstrap@db60ae0 in a separate PR because it is complicated to modify and so probably to review |
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.
Looks really great compared to the number of commits implemented here. Some few changes before seal of approval !
SonarCloud Quality Gate failed. 0 Bugs No Coverage information |
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 ! Great work 🎉
First batch of commits to integrate into our main branch before Add dark mode support (which will be a dedicated PR) and then the commits after (in another dedicated PR right after).
twbs/bootstrap@bf6240d: nothing to do here because we don't have floating labelstwbs/bootstrap@01dc2f5: nothing to do here, this commit was already gathered for the v5.2.3twbs/bootstrap@8291746: nothing to do here, we don't have these examples in Boosted anymoretwbs/bootstrap@5029370: nothing to do here, this commit was already gathered for the v5.2.3twbs/bootstrap@4822984: nothing to do here, we don't have the cover example in Boostedtwbs/bootstrap@39bbca5: don't regeneratepackage-lock.json
right nowtwbs/bootstrap@bbe1ffd: nothing to do here since our navbars example is different from Bootstrap's onetwbs/bootstrap@06bed9b: nothing to do here since we don't have floating labelstwbs/bootstrap@2b21094: nothing to do here, this commit was already gathered for the v5.2.3twbs/bootstrap@3e50a79: nothing to do here, the bundlewatch is already correct at this pointtwbs/bootstrap@8387827: nothing to do cause we don't have any codeql workflowtwbs/bootstrap@9aff4cd: nothing to do cause we don't have any codeql workflowtwbs/bootstrap@8576521: nothing to do here cause we don't have floating labels in BoostedIn this section, the commits are in fact a merge back from Bootstrap v5.2.3, so already gathered in commits before
twbs/bootstrap@b722369twbs/bootstrap@77c456dtwbs/bootstrap@127a816twbs/bootstrap@7e3074ctwbs/bootstrap@3958947twbs/bootstrap@cb02143twbs/bootstrap@8f6de23twbs/bootstrap@c139bactwbs/bootstrap@a1a9895: nothing to do cause we don't have the same navbars example in Boosted