8000 Fix +mixin style include results in "TypeError: Cannot read property 'first' of undefined" by dotdat · Pull Request #422 · sasstools/sass-lint · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix +mixin style include results in "TypeError: Cannot read property 'first' of undefined" #422

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

Merged
merged 4 commits into from
Dec 28, 2015
Merged

Fix +mixin style include results in "TypeError: Cannot read property 'first' of undefined" #422

merged 4 commits into from
Dec 28, 2015

Conversation

dotdat
Copy link
Contributor
@dotdat dotdat commented Nov 20, 2015

DCO 1.1 Signed-off-by: Brian George bpgeorge88@gmail.com

Sorry it took me so long to get this back into review.

@benthemonkey
Copy link
Member

Is there a GitHub issue that this fixes? If so, can you include a link to it?

@dotdat
Copy link
Contributor Author
dotdat commented Nov 20, 2015

Sure thing: #380

@benthemonkey
Copy link
Member

LGTM but I'll wait for someone else to look too.


.foo-noextend
+foo
content: "bar"
Copy link
Member

Choose a reason for hiding this comment

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

Missing new line at end of file

@DanPurdy
Copy link
Member

Hi @dotdat no problem, thank you for taking the time to fix this issue. Two extremely minor issues but would be good to add them in as it's defined in our editorconfig that we require newlines at the ends of files. Do you use an IDE with editorconfig support?

This is good to merge once it's updated. Thanks again!

@dotdat
Copy link
Contributor Author
dotdat commented Dec 28, 2015

Hey @DanPurdy -- sorry for the massive delay, the last month left me with 0 free time. I've updated this according to the editorconfig specs and just wanted to double check back before I did anything. Thanks!

@DanPurdy
Copy link
Member

Hi @dotdat, no problem I think most of us have been in the same boat recently! Thanks for taking the time here looks good.

On a side note you may just want to add the email address that you just used to do those last commits as a secondary email for your github account to get the proper contribution numbers.

😄

@DanPurdy DanPurdy added this to the 1.5.0 milestone Dec 28, 2015
DanPurdy added a commit that referenced this pull request Dec 28, 2015
Fix +mixin style include results in "TypeError: Cannot read property 'first' of undefined"
@DanPurdy DanPurdy merged commit 3ab3b84 into sasstools:develop Dec 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0