-
Notifications
You must be signed in to change notification settings - Fork 522
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
Fix +mixin style include results in "TypeError: Cannot read property 'first' of undefined" #422
Conversation
Is there a GitHub issue that this fixes? If so, can you include a link to it? |
Sure thing: #380 |
LGTM but I'll wait for someone else to look too. |
|
||
.foo-noextend | ||
+foo | ||
content: "bar" |
There was a problem hiding this comment.
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
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! |
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! |
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. 😄 |
Fix +mixin style include results in "TypeError: Cannot read property 'first' of undefined"
DCO 1.1 Signed-off-by: Brian George bpgeorge88@gmail.com
Sorry it took me so long to get this back into review.