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

Update specs to keep trailing comments on the previous line #1485

Merged
merged 3 commits into from Jun 3, 2022
Merged

Conversation

nex3
Copy link
Contributor

@nex3 nex3 commented Oct 16, 2019

See sass/dart-sass#849
[skip dart-sass]

@nex3 nex3 requested a review from nickbehrens October 16, 2019 21:23
spec/css/keyframes.hrx Outdated Show resolved Hide resolved
@nex3 nex3 removed the request for review from nickbehrens June 1, 2022 20:11
@nex3
Copy link
Contributor Author

nex3 commented Jun 1, 2022

I can't approve this because I'm the one who originally opened it, but 👍.

@Goodwine
Copy link
Member

Goodwine commented Jun 1, 2022

I discovered I could approve it, I'll wait until the other PR is ready so they go in at the same time

Goodwine added a commit to sass/dart-sass that referenced this pull request Jun 3, 2022
See sass/sass-spec#1485

- Update lib/src/visitor/serialize.dart to stop using old-style int-based for loop.
- Extend FileSpan with a .contains(targetSpan) method

Co-authored-by:  Nick Behrens <nbehrens@google.com>
Co-authored-by:  Carlos Israel Ortiz García <goodwine@google.com>
Co-Authored-By: Natalie Weizenbaum <nweiz@google.com>
@Goodwine Goodwine merged commit af752b0 into main Jun 3, 2022
@Goodwine Goodwine deleted the fix-417 branch June 3, 2022 01:46
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.

None yet

2 participants