Skip to content
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

使用dart-sass构建失败 #158

Closed
Bubbu0129 opened this issue Oct 31, 2024 · 5 comments · Fixed by #160
Closed

使用dart-sass构建失败 #158

Bubbu0129 opened this issue Oct 31, 2024 · 5 comments · Fixed by #160
Assignees
Labels
bug ISSUE: Something isn't working

Comments

@Bubbu0129
Copy link
Contributor

Info

Arch Linux
dart-sass 1.79.4

Issue

根据贡献指南安装dart-sass后,运行make报错。
Makefile, lines 21-25中的scss --sourcemap=none改为sass --no-source-map可以解决此问题;前者似乎是ruby-sass的写法。
编译时,也存在多个Warning:

Deprecation Warning: As of Dart Sass 2.0.0, !global assignments won't be able to declare new variables.
Recommendation: add `$<variable>: null` at the stylesheet root.

似乎是dart-sass兼容问题。

@Bubbu0129 Bubbu0129 added the bug ISSUE: Something isn't working label Oct 31, 2024
@RalXYZ
Copy link
Collaborator

RalXYZ commented Oct 31, 2024

@Bubbu0129 非常感谢您的反馈。
贡献文档中的内容与 CI/CD 的环境不对齐。在本项目的 CI/CD 环境中,我们采用的是 ruby sass。这是不好的。

sudo apt install ruby sass -y

sudo apt install ruby sass -y

我觉得我们现在应该彻底解决这一问题,包括:

  1. 确保项目的 CI/CD 使用 https://www.npmjs.com/package/sass ,即 dart sass。与 CI/CD 相关的配置文件,仅有上面引用的两个文件。
  2. 基于 1,检查编译过程中出现的全部警告、错误,并全部予以解决。

如果您愿意完成以上两点工作内容,在工作完成后,相关的 pr 即可被合入主线。如果您无暇跟进,我们会基于您的工作完成上述两点工作内容,并给您合适的 credit ;随后相关的 pr 也会被合入主线。
无论如何,再次感谢您。

@Bubbu0129
Copy link
Contributor Author

  1. 项目的CI/CD通过安装 GitHub release asset tarball 的方式迁移到 dart sass。dev-build action在我的fork上测试成功。
  2. 针对 dart sass,已经解决了本地编译的全部警告和问题。

PS: 我并不熟悉 GitHub Action,如有错误请多包涵。在改动workflow的时候,相比于配置 node.js 环境后再使用 npm 安装 Dart Sass,我认为直接从 sass/dart-sass Release 中下载二进制文件更为直接。

@RalXYZ
Copy link
Collaborator

RalXYZ commented Nov 1, 2024

非常感谢。我批准了 #160 的 CI (Dev Build #162) ,构建成功,且构建结果符合预期。

我在 #160 留下了一点 comment,辛苦您按照其中的内容进行微调。之所以提出这一更改,除了出于安全性考虑,也是因为发生了 #161 中提及的事情(#161 里提及的内容我后面会自行予以修复)

@Bubbu0129
Copy link
Contributor Author

已将 dart sass 版本固定到 1.79.6。#161 解决后,可以再将 dart sass 升级到 1.80。
希望您在 wiki 贡献指南中指明 dart sass 的版本;同时,我在调试的时候触发了多次 workflow,问题解决后您可以把失败的 workflow run 删除。

@RalXYZ
Copy link
Collaborator

RalXYZ commented Nov 2, 2024

同时,我在调试的时候触发了多次 workflow,问题解决后您可以把失败的 workflow run 删除。

感谢提醒,这个我觉得也不必删了

再次感谢您的贡献和配合

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ISSUE: Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
@RalXYZ @Bubbu0129 and others