alamb
commented
6 months ago I tested this locally and it seems to work well
The only issue I saw was some new warnings about deprecation
eprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($spacer, 2) or calc($spacer / 2)
More info and automated migrator: https://sass-lang.com/d/slash-div
╷
302 │ $headings-margin-bottom: $spacer / 2 !default;
│ ^^^^^^^^^^^
╵
/arrow-site/node_modules/bootstrap/scss/_variables.scss 302:31 @import
bootstrap/scss/bootstrap.scss 9:9 @import
/arrow-site/css/main.scss 1:9 root stylesheet
Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($input-padding-y, 2) or calc($input-padding-y / 2)
More info and automated migrator: https://sass-lang.com/d/slash-div
╷
498 │ $input-height-inner-quarter: add($input-line-height * .25em, $input-padding-y / 2) !default;
│ ^^^^^^^^^^^^^^^^^^^^
╵
/arrow-site/node_modules/bootstrap/scss/_variables.scss 498:73 @import
bootstrap/scss/bootstrap.scss 9:9 @import
/arrow-site/css/main.scss 1:9 root stylesheet
Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($custom-control-indicator-size, 2) or calc($custom-control-indicator-size / 2)
More info and automated migrator: https://sass-lang.com/d/slash-div
╷
568 │ $custom-switch-indicator-border-radius: $custom-control-indicator-size / 2 !default;
│ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
╵
/arrow-site/node_modules/bootstrap/scss/_variables.scss 568:49 @import
bootstrap/scss/bootstrap.scss 9:9 @import
/arrow-site/css/main.scss 1:9 root stylesheet
Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($spacer, 2) or calc($spacer / 2)
More info and automated migrator: https://sass-lang.com/d/slash-div
╷
713 │ $nav-divider-margin-y: $spacer / 2 !default;
│ ^^^^^^^^^^^
╵
/arrow-site/node_modules/bootstrap/scss/_variables.scss 713:37 @import
bootstrap/scss/bootstrap.scss 9:9 @import
/arrow-site/css/main.scss 1:9 root stylesheet
Deprecation Warning: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.
Recommendation: math.div($spacer, 2) or calc($spacer / 2)
More info and automated migrator: https://sass-lang.com/d/slash-div
╷
718 │ $navbar-padding-y: $spacer / 2 !default;
│ ^^^^^^^^^^^
╵
/arrow-site/node_modules/bootstrap/scss/_variables.scss 718:37 @import
bootstrap/scss/bootstrap.scss 9:9 @import
/arrow-site/css/main.scss 1:9 root stylesheet
Warning: 64 repetitive deprecation warnings omitted.
Run in verbose mode to see all warnings.
As suggested by @kou in https://github.com/apache/arrow-site/pull/448#issuecomment-1878591082 let's use the latest Jekyll so the docs build with the latest ruby version
Note that @bkmgit has another PR that updates all dependencies in https://github.com/apache/arrow-site/pull/449