-
Notifications
You must be signed in to change notification settings - Fork 1
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
Test: Temporarily disable all visual tests and SCSS loading #1581
Conversation
✅ Deploy Preview for spirit-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I think you also want to exclude the branch from the CI runs:
|
426facb
to
49f78a7
Compare
I fixed the build, do you still think I need it? Now I don't see why, but maybe I miss something :) |
I thought that you did not want to run in on CI in this case. 🤷 |
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.
🙏🏻 Please, drop a comment every time you comment out a code, and ideally file a cleanup issue for it. It's always a good practice to do so because there is no guarantee we won't change our focus and finish what we do now.
1e4a2b4
to
3619639
Compare
49f78a7
to
c71b4a4
Compare
c71b4a4
to
703c5cd
Compare
We need this because everything is broken now.
The plan is to slowly uncomment/unignore everything as we transform components into new tokens.
Description
Additional context
Issue reference