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

WIP: Upgrade addon and dependencies #242

Closed
wants to merge 4 commits into from

Conversation

bjornharrtell
Copy link

@bjornharrtell bjornharrtell commented Jun 11, 2019

This is work in progress. Remaining:

  • Resolve error Failed to find 'basscss'
  • Replace postcss-cssnext with postcss-preset-env

@ynnoj is this effort welcome? Also I note you might have some sensitive info at .travis.yml#L64 ... I opted to not include that in the conflict resolution during upgrade.

@mansona
Copy link
Owner

mansona commented Nov 12, 2019

Hey @bjornharrtell thanks for your contribution 👍

I've updated the dependencies in #251 (comment) so it will get released very soon. Also we're not going to be swapping from npm to yarn at this stage.

Let me know if there is something that is missing from the new PR 👍

@mansona mansona closed this Nov 12, 2019
@bjornharrtell bjornharrtell deleted the up-deps branch November 12, 2019 07:12
@bjornharrtell
Copy link
Author

Great news @mansona, thanks!

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.

2 participants