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

Rewrite much of argument handling. #113

Merged
merged 7 commits into from
Jun 20, 2021
Merged

Rewrite much of argument handling. #113

merged 7 commits into from
Jun 20, 2021

Conversation

kaj
Copy link
Owner

@kaj kaj commented Jun 4, 2021

No description provided.

@kaj kaj force-pushed the rework-arguments branch 10 times, most recently from 9356fca to 6f21a24 Compare June 9, 2021 20:13
@kaj kaj force-pushed the rework-arguments branch 2 times, most recently from c14e7f8 to 41ec20a Compare June 19, 2021 20:42
@kaj kaj merged commit 7d2fab6 into master Jun 20, 2021
@kaj kaj deleted the rework-arguments branch June 20, 2021 13:13
@kaj kaj mentioned this pull request Jun 20, 2021
kaj added a commit that referenced this pull request Jun 20, 2021
Progress: 3992 of 6180 tests passed in dart-sass compatibility mode.

* The `sass::Item::MixinCall` enum alternative was modified.
* Signature changes in `css::CallArgs` and `sass:CallArgs`,
  and new alternatives in `sass::ArgsError`.
* `css::Value` has a new variant `ArgList`.
* The `OrderMap::keys()` and `values()` functions now returns iterators,
  `OrderMap::remove(key)` now returns the removed value.

* Better handling of mixin call bodies, the `@content` item, and the
  `content_exists` function.  Fixes #112.
* `@import` exposes things `@forward`ed in the imported module.
* Many improvements in parameter passing to functions and mixins.  PR #113.
* Some more fixes regarding function calls and modules.
* Support the `keywords` function in `sass:meta`.
* Removed some debug printouts that was accidentally left in 0.21.0.
* Update sass-spec test suite to 2021-06-14.

Thanks to @Keats for reporting #112 and #108.
@kaj kaj mentioned this pull request Mar 5, 2022
17 tasks
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.

1 participant