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

Update jest, typescript, babel, eslint-config-seek and minimum node version requirement #709

Merged
merged 14 commits into from
Oct 11, 2022

Conversation

askoufis
Copy link
Contributor

@askoufis askoufis commented Oct 7, 2022

See changesets.

Note: eslint was not updated because eslint-plugin-cypress is not compatible with eslint 8. It's being moved to the cypress repo soon, so will hopefully be updated to support after that's complete.

@askoufis askoufis requested a review from a team as a code owner October 7, 2022 05:02
@changeset-bot
Copy link

changeset-bot bot commented Oct 7, 2022

🦋 Changeset detected

Latest commit: f3fde7c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
sku Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mrm007
Copy link
Contributor

mrm007 commented Oct 9, 2022

Looks like one snapshot needs to be updated.

Unrelated, seems like we're running Validate / Lint & Test on both pull_request and push. I've seen this solved with

on:
  pull_request:
  push:
    branches: [master]

with the caveat it doesn't run on push before opening a PR. I don't necessarily see this as a downside — if you want that check to run, open a draft PR.

@askoufis askoufis changed the title Update jest, eslint-config-seek and minimum node version requirement Update jest, typescript, eslint-config-seek and minimum node version requirement Oct 10, 2022
@@ -152,7 +153,7 @@
"traverse": "^0.6.6",
"treat": "^2.0.4",
"tree-kill": "^1.2.1",
"typescript": ">=4.1.3 <4.5.0",
"typescript": "^4.5.0",
Copy link
Contributor

@mrm007 mrm007 Oct 10, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This goes straight to latest, 4.8.4. Do we want to use ~4.5.0 to stay on 4.5.x?

sku/yarn.lock

Lines 16975 to 16976 in b03adb4

typescript@^4.5.0:
version "4.8.4"

Copy link
Contributor Author

@askoufis askoufis Oct 10, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skuba does this, though I'm not exactly sure of the benefit in the case of sku. If anything, I feel like we'd want to restrict the upper limit, since typescript doesn't adhere to semver, just in case something breaks, but I'm not sure about restricting to a specific minor version as that feels a bit too limiting.

@askoufis askoufis changed the title Update jest, typescript, eslint-config-seek and minimum node version requirement Update jest, typescript, babel, eslint-config-seek and minimum node version requirement Oct 11, 2022
Copy link
Contributor

@michaeltaranto michaeltaranto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏

@askoufis askoufis merged commit 7a5d2a8 into master Oct 11, 2022
@askoufis askoufis deleted the jest-eslint-node-updates branch October 11, 2022 03:40
@seek-oss-ci seek-oss-ci mentioned this pull request Oct 11, 2022
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.

3 participants