-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
add first class mixins to embedded sass protobuf and JS API #3674
Conversation
We should bump the version number in https://github.com/sass/sass/blob/main/spec/EMBEDDED_PROTOCOL_VERSION |
As this creates a new kind of value that can be sent to the host, is it actually a minor version or a major one ? |
Historically this kind of change has been considered as minor. The host can upgrade to new protocol version without supporting added type, and all previous feature would still work. |
26a6ae5
to
f326753
Compare
f326753
to
8998009
Compare
This looks good! Only thing is that we should probably edit the title of this PR as the scope of this PR has increased. |
* main: Remove `isAlphaMissing` and add "alpha" to channel name types. Rename types using title-case for acronyms longer than two letters in camel-case identifiers. Clarifying performance expectations about sass and sass-embedded (sass#3716) Tweak language around getters and arrays. Code review Remove unused ValueObject type [First-Class Mixins] Flush to spec Move and restructure how we specify scopes Fix typos in the deprecation APi proposal (sass#3708) Return immutable types Remove generic change overload Add first class mixins to embedded sass protobuf and JS API (sass#3674) Draft 1.1 changes for Color spaces JS API (sass#3704) Add support for the relative color syntax from CSS Color 5 (sass#3676) Bump tj-actions/changed-files from 39.0.3 to 39.2.0 (sass#3697) Package Importer updates (sass#3699) Fix link in Contributing docs (sass#3705)
See #3626 and sass/dart-sass#2073