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

docs: recommend [email protected] #1658

Merged
merged 1 commit into from
Oct 18, 2016
Merged

docs: recommend [email protected] #1658

merged 1 commit into from
Oct 18, 2016

Conversation

jacquerie
Copy link
Contributor

@jacquerie jacquerie commented Oct 18, 2016

More recent versions of node-sass cause issue #1642, therefore
recommend developers to use this particular version (closes #1644).

Closes #1655

@zzacharo: can you run

git config --local user.name "Zacharias Zacharodimos"
git config --local user.email <[email protected]>

so that your local git configuration matches what I put for you here?

* More recent versions of `node-sass` cause issue #1642, therefore
  recommend developers to use this particular version (closes #1644).

Signed-off-by: Zacharias Zacharodimos <[email protected]>
@jacquerie jacquerie merged commit 9d15679 into inspirehep:master Oct 18, 2016
@jacquerie jacquerie deleted the integrate-pr-1655 branch October 18, 2016 11:57
@zzacharo
Copy link
Contributor

@jacquerie I changed it. You changed the commit message for me?

@jacquerie
Copy link
Contributor Author

Yup, and I already merged it!

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.

docs: document the pin on node-sass
2 participants