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

Add tests for quote #1380

Merged
merged 1 commit into from
Apr 14, 2019
Merged

Add tests for quote #1380

merged 1 commit into from
Apr 14, 2019

Conversation

glebm
Copy link
Contributor

@glebm glebm commented Apr 13, 2019

Refs sass/libsass#2811

[skip libsass]

libsass PR: sass/libsass#2873

@glebm glebm requested a review from nex3 April 13, 2019 11:28
glebm added a commit to glebm/libsass that referenced this pull request Apr 13, 2019
glebm added a commit to glebm/libsass that referenced this pull request Apr 13, 2019
glebm added a commit to glebm/libsass that referenced this pull request Apr 14, 2019
@mgreter mgreter merged commit 87d9b00 into master Apr 14, 2019
@glebm
Copy link
Contributor Author

glebm commented Apr 14, 2019

In the future, the libsass PR should be merged first so we don't break the libsass build

glebm added a commit to sass/libsass that referenced this pull request Apr 15, 2019
@glebm glebm deleted the quote branch April 15, 2019 06:49
Copy link
Contributor

@nex3 nex3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should be spec/core_functions/string/quote.hrx (as per DO organize specs by which part of the language they test).

@mgreter Please also don't merge PRs before the assigned reviewer has a chance to review, especially while we're all still getting used to the style guide.

spec/string-functions/quote.hrx Show resolved Hide resolved
spec/string-functions/quote.hrx Show resolved Hide resolved
spec/string-functions/quote.hrx Show resolved Hide resolved
glebm added a commit that referenced this pull request Apr 19, 2019
glebm added a commit that referenced this pull request Apr 19, 2019
glebm added a commit that referenced this pull request Apr 19, 2019
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.

3 participants