Skip to content
This repository has been archived by the owner on Sep 21, 2023. It is now read-only.

ci: Use 9 instead of stable node #414

Merged
merged 1 commit into from
Apr 25, 2018
Merged

ci: Use 9 instead of stable node #414

merged 1 commit into from
Apr 25, 2018

Conversation

misund
Copy link
Contributor

@misund misund commented Apr 25, 2018

We indirectly depend on anodynos/upath via webpack. upath specifically states it does not support node > 9.

Stable should be added back in as soon as our dependency graph has updated to address this – hopefully it will not be too long.

See anodynos/upath#14
See anodynos/upath#15

Please tick a box

  • ci (Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs))

If you're adding a feature, is it documented in a storybook story?

  • Not applicable

Are the components you're working on reusable between brands?

  • Not applicable

If you're creating markup, did you add proper semantics?

  • Not applicable

(Did you do a CR and see that there is something that we should check for each PR, that are not on the list, please update this document)

We indirectly depend on anodynos/upath via webpack. upath specifically states it does not support node > 9.

Stable should be added back in as soon as our dependency graph has updated to address this – hopefully it will not be too long.

See anodynos/upath#14
See anodynos/upath#15
@misund misund requested a review from lupakivan April 25, 2018 10:34
@misund misund merged commit dd66630 into master Apr 25, 2018
@misund misund deleted the ci/9-for-stable branch April 25, 2018 10:42
misund pushed a commit that referenced this pull request Apr 26, 2018
* Removed useless code

* Code refactoring

* Refactoring

* docs(readme): Fix typos

* ci(percy): Run percy on master/node 9; see #414, see #415
@anodynos
Copy link

Hello everyone - I merged anodynos/upath#15 & published it - thanks & sorry for the delay! You can update to node@10, once you check its working.

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.

3 participants