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

chore(deps-dev): bump typescript from 4.8.4 to 4.9.3 #1019

Merged
merged 6 commits into from
Nov 21, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 16, 2022

Bumps typescript from 4.8.4 to 4.9.3.

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added javascript Pull requests that update Javascript code ⛓️ dependencies Pull requests that update a dependency file labels Nov 16, 2022
@codecov
Copy link

codecov bot commented Nov 16, 2022

Codecov Report

Merging #1019 (f6c7424) into master (1c1d098) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1019   +/-   ##
=======================================
  Coverage   99.61%   99.61%           
=======================================
  Files          61       61           
  Lines        1041     1041           
  Branches      163      163           
=======================================
  Hits         1037     1037           
  Misses          2        2           
  Partials        2        2           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ArttuOll
Copy link
Contributor

I will investigate this tomorrow.

@ArttuOll
Copy link
Contributor

ArttuOll commented Nov 17, 2022

Relates to microsoft/TypeScript#51567 .

We're getting hit by outdated types in our dependencies. This is fixed by recreating the lockfile.

After that we're getting hit by the correctness updates to lib.d.ts, but luckily only in one mock function.

@ArttuOll ArttuOll requested a review from a team November 17, 2022 16:10
@ArttuOll ArttuOll marked this pull request as draft November 17, 2022 16:16
@ArttuOll
Copy link
Contributor

And now our linter goes insane... Perhaps I should consider a more elegant approach than just regenerating the lockfile.

@ArttuOll
Copy link
Contributor

Installing @types/node seems to be the best solution here.

Also, during linting, this message now appears:

WARNING: You are currently running a version of TypeScript which is not officially supported by @typescript-eslint/typescript-estree.

You may find that it works just fine, or you may not.

SUPPORTED TYPESCRIPT VERSIONS: >=3.3.1 <4.9.0

YOUR TYPESCRIPT VERSION: 4.9.3

Please only submit bug reports when using the officially supported version.

Does not seem too bad to me.

What does everybody think about this upgrade?

@ArttuOll ArttuOll marked this pull request as ready for review November 17, 2022 16:31
@ArttuOll ArttuOll requested a review from a team November 17, 2022 16:31
dependabot bot and others added 5 commits November 21, 2022 20:11
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.8.4 to 4.9.3.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Commits](https://github.com/Microsoft/TypeScript/commits)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
This relates to the release of TypeScript 4.9. The correctness fixes related to lib.d.ts which were
a part of this release broke the parameter type of this mock-function.
This relates to the release of TypeScript 4.9, which broke our build. In the corresponding issue in
the TypeScript repo, multiple solutions were proposed of which installing @types/node seems to work
the best.
@xobotyi xobotyi force-pushed the dependabot/npm_and_yarn/typescript-4.9.3 branch from 250a684 to 4dcae8a Compare November 21, 2022 19:13
@xobotyi
Copy link
Contributor

xobotyi commented Nov 21, 2022

there were no need to add @types/node, the only thing required - stub types fix.

@ArttuOll
Copy link
Contributor

My first solution had that, but then the linter went insane. The same thing that we're seeing now.

@xobotyi
Copy link
Contributor

xobotyi commented Nov 21, 2022

thats just a matter of changed types of testing library - im disabling false-positive linter for tests

That linter raised false-positives errors
@xobotyi xobotyi merged commit 355e569 into master Nov 21, 2022
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/typescript-4.9.3 branch November 21, 2022 19:24
@xobotyi
Copy link
Contributor

xobotyi commented Dec 2, 2022

🎉 This PR is included in version 20.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛓️ dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code released
Development

Successfully merging this pull request may close these issues.

2 participants