Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

Add workaround for yarn node-sass bug #498

Merged
merged 1 commit into from
Jan 20, 2017

Conversation

acburdine
Copy link
Member

Workaround for yarnpkg/yarn#1981

Found in this commit: tjgrathwell/ng-picross@22d55c0

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.069% when pulling 0ee3240 on acburdine:dammit-yarn into 7652969 on TryGhost:master.

@kevinansfield
Copy link
Member

👍 let's see how it goes

@kevinansfield kevinansfield merged commit 1556004 into TryGhost:master Jan 20, 2017
@Headcult
Copy link

Same issue here, while using Yarn (0.21.3). It's failing at Heroku build (node-sass error). Even the suggested workaround did not work for me (before_script: - yarn add --force node-sass). Can we get a ETA on the fix for this as Travis CI is failing at Deploy stage due to Heroku build failure. Thank you.

@kevinansfield
Copy link
Member

@Headcult we merged the fix in this PR on the 20th Jan and we haven't had any further issues with Travis failing to build Ghost-Admin. I understand you are having trouble but this is not the right repo to chase yarn issues, I see you've already posted the same message on the associated yarn issue - if you can keep the discussion there it will prevent any confusion by starting discussions in an unrelated project 😄

@Headcult
Copy link

Sorry @kevinansfield , repeated the issue here by mistake. Yes, I'll track this with Yarn through the other issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants