-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Fix default value for $enable-deprecation-messages
#32940
Conversation
As seen in the https://github.com/twbs/bootstrap/blob/main/scss/_variables.scss, the $enable-deprecation-messages variable (line 220) is set to `true` by default. Direct permalink to the line 220 @ GitHub: https://github.com/twbs/bootstrap/blob/e79c8f3489527d7f5eef2bb3cf14856f26c49871/scss/_variables.scss#L220
Weird, it seems we have the same in v4-dev @mdo. |
$enable-deprecation-messages
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.
@XhmikosR You're right, the same in v4.6.
Left another comment regarding version: v5 should mention deprecation for v6, shouldn't it?
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.
With @ffoodd's suggested change, this LGTM I think.
Co-authored-by: Gaël Poupard <[email protected]>
* Update the default value for $enable-deprecation-messages As seen in the https://github.com/twbs/bootstrap/blob/v4-dev/scss/_variables.scss, the `$enable-deprecation-messages` variable is set to `true` by default.
* Update the default value for $enable-deprecation-messages As seen in the https://github.com/twbs/bootstrap/blob/v4-dev/scss/_variables.scss, the `$enable-deprecation-messages` variable is set to `true` by default.
* Update the default value for $enable-deprecation-messages As seen in the https://github.com/twbs/bootstrap/blob/v4-dev/scss/_variables.scss, the `$enable-deprecation-messages` variable is set to `true` by default.
As seen in the https://github.com/twbs/bootstrap/blob/main/scss/_variables.scss, the
$enable-deprecation-messages
variable (line 220) is set totrue
by default.Direct permalink to the line 220 @ GitHub:
bootstrap/scss/_variables.scss
Line 220 in e79c8f3