Skip to content

Conversation

@illusionfield
Copy link
Contributor

Summary

This PR introduces fourseven:scss@5.0.0-beta.1, which completes the migration to Dart Sass and officially removes the built-in sass dependency.

It combines the changes from two previous PRs into one consolidated update targeting the release/5.0.0 branch.


What's Changed

  • 💥 Breaking: sass is no longer bundled — users must now install either sass or sass-embedded manually via npm
  • 🚀 Flexibility: Developers can now pin versions or switch Sass implementations without modifying this plugin
  • 📚 Docs:
    • README has been restructured and clarified to reflect the new usage model
    • Migration Guide expanded and updated, now includes:
      • Manual Sass installation
      • Sass engine flexibility
      • PostCSS setup
      • Modern @use/@forward transition instructions
  • 🧼 Removed legacy references to node-sass and outdated patterns
  • 🧪 Debug mode and custom config options remain fully supported

Migration Notes

To upgrade to this version, you must manually install a Sass compiler:

meteor npm install --save-dev sass
# or
meteor npm install --save-dev sass-embedded

For more details, see the Migration Guide


Why This Matters

  • Keeps the package leaner and less opinionated
  • Unlocks performance with sass-embedded (native binary)
  • Makes the plugin more robust for future Meteor 3 releases
  • Easier for community contributors to reason about dependencies

Resolved Issues


This replaces and supersedes the two earlier PRs (now closed):

illusionfield and others added 29 commits February 7, 2025 15:32
* master:
  Internal Meteor Packages: improve root file detection logic for imports and package files Meteor-Community-Packages#324
Addressed an issue where running `meteor test-packages` in Meteor 2 failed due to npm dependency installation errors related to the `meteor-node-stubs` update. This fix ensures compatibility and resolves the 'npmignore: command not found' error during package tests.

For more details, see: meteor/meteor#13691
- Removed `sass` as a direct dependency to allow full control over Sass versioning
- Enables usage of both `sass` and `sass-embedded` via devDependencies
- Updated README.md for improved clarity, examples, and migration guidance
- Bumped version to `5.1.0-beta.1` to reflect breaking internal change
@illusionfield
Copy link
Contributor Author

@StorytellerCZ I'm done with the restructuring — I didn’t see a release/5.0.0 branch, so I targeted release-5 for now.
We may want to clean up the release branches a bit later.
I’ve also linked the issues that can be closed as part of this.

@jankapunkt 5.0.0-beta.1 is ready for release whenever you are.

@illusionfield
Copy link
Contributor Author

Hi @jankapunkt @StorytellerCZ 👋

Just following up to check in — any updates on this?

5.0.0-beta.1 is ready and waiting, and I’d love to get it out so the community can start testing and giving feedback.

Let me know if there's anything else you need from my side!

@StorytellerCZ
Copy link
Member

Published fourseven:scss@5.0.0-beta.1.

@StorytellerCZ
Copy link
Member

OK, I think we had enough in beta. I'm going to move to RC today to give more people a push to test it.

@jankapunkt
Copy link
Member

Please everyone keep testing and integrating, we will do a release on 1st December (unless someone detects bigger issues).

- replace @zodern/mtest with @illusionfield/mtest and silence verbose output
- new fork is the maintained path, so we keep tests running reliably
@jankapunkt
Copy link
Member

A bit of delay but I am about to release during this week

@jankapunkt jankapunkt merged commit 6cacc9e into Meteor-Community-Packages:release/5 Dec 15, 2025
3 checks passed
@illusionfield
Copy link
Contributor Author

@jankapunkt
This will need to be rewritten.

> If you are upgrading from an earlier version, please read the [Migration Guide](https://github.com/illusionfield/fourseven-scss/blob/master/MIGRATION-GUIDE-v5.md), as this README does not cover all necessary migration steps.

@jankapunkt
Copy link
Member

@illusionfield do you have a prompt I can forward to copilot?

@illusionfield
Copy link
Contributor Author

illusionfield commented Dec 15, 2025

@jankapunkt
In README.md please update the Migration Guide link from:

https://github.com/illusionfield/fourseven-scss/blob/master/MIGRATION-GUIDE-v5.md

to:

https://github.com/Meteor-Community-Packages/meteor-scss/blob/master/MIGRATION-GUIDE-v5.md

It doesn’t need to point to my fork anymore, since it has been merged into the main repo.

@illusionfield illusionfield deleted the release/5.0.0 branch December 15, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fourseven:scss@5.0.0-alpha.2 - import bug Issue with fourseven:scss@5.0.0-alpha.1 and Internal Meteor Packages

3 participants