components: help(Upgrade): Unable to finish upgrade from 12 to 13 due to scss issues
Is this a regression?
- Yes, this behavior used to work in the previous version
The previous version in which this bug was not present was
12
Description
When upgrading from angular 12 to angular 13 of the material library the upgrade completes successfully but the build fails with cryptic error message.
Reproduction
Steps to reproduce:
- clone https://github.com/IsraelHikingMap/Site
- cd IsraelHiking.Web
- upgrade angular from 12 to 13
- upgrade material from 12 to 13
- run npm run build:watch
- Build fails with the following error message:
SassError: expected "{".
╷
1 │ @use "@angular/material" as mat;
│ ^
╵
src/scss/sidebar.scss 1:34 @import
src/scss/common.scss 6:9 root stylesheet
Expected Behavior
Upgrade should complete successfully and build should pass.
Actual Behavior
Unable to use material 13 due to this cryptic error which I haven’t found a way to workaround or fix.
Environment
- Angular: 13
- CDK/Material: 13
- Browser(s): N/A
- Operating System (e.g. Windows, macOS, Ubuntu): maxOS
About this issue
- Original URL
- State: closed
- Created 2 years ago
- Reactions: 2
- Comments: 25 (7 by maintainers)
Commits related to this issue
- fix(cdk/schematics): strip bom from sass files We have some code to strip BOM from TS files when running a migration, but it doesn't apply to migrations of Sass files. These changes reuse the same c... — committed to crisbeto/material2 by crisbeto 2 years ago
- fix(cdk/schematics): strip bom from sass files We have some code to strip BOM from TS files when running a migration, but it doesn't apply to migrations of Sass files. These changes reuse the same c... — committed to crisbeto/material2 by crisbeto 2 years ago
- fix(cdk/schematics): strip bom from sass files We have some code to strip BOM from TS files when running a migration, but it doesn't apply to migrations of Sass files. These changes reuse the same c... — committed to crisbeto/material2 by crisbeto 2 years ago
- fix(cdk/schematics): strip bom from sass files (#25364) We have some code to strip BOM from TS files when running a migration, but it doesn't apply to migrations of Sass files. These changes reuse t... — committed to angular/components by crisbeto 2 years ago
- fix(cdk/schematics): strip bom from sass files (#25364) We have some code to strip BOM from TS files when running a migration, but it doesn't apply to migrations of Sass files. These changes reuse t... — committed to angular/components by crisbeto 2 years ago
- fix(cdk/schematics): strip bom from sass files (#25364) We have some code to strip BOM from TS files when running a migration, but it doesn't apply to migrations of Sass files. These changes reuse t... — committed to angular/components by crisbeto 2 years ago
I’m still stuck with material 12 due to this issue, I have not found a solution unfortunately…