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

build(deps): bump got from 13.0.0 to 14.2.1 #1197

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 20, 2024

Bumps got from 13.0.0 to 14.2.1.

Release notes

Sourced from got's releases.

v14.2.1

  • Fix error handling when UTF-8 decoding fails (#2336) c81a611

sindresorhus/got@v14.2.0...v14.2.1

v14.2.0

  • Add cause property with the original error to RequestError (#2327) 4cbd01d

sindresorhus/got@v14.1.0...v14.2.0

v14.1.0

  • Allow typing the body of a RequestError response (#2325) 5e4f6ff

sindresorhus/got@v14.0.0...v14.1.0

v14.0.0

Breaking

  • Require Node.js 20 (#2313) a004263
    • Why not target the oldest active Node.js LTS, which is Node.js 18? I usually strictly follow this convention in my packages. However, this package is the exception because the HTTP part of Node.js is consistently buggy, and I don't have time to work around issues in older Node.js releases. I you need to still support Node.js 18, I suggest staying on Got v13, which is quite stable. Node.js 18 will be out of active LTS in 5 months.

Improvements

  • Make followRedirect option accept a function (#2306) 7c3f147

sindresorhus/got@v13.0.0...v14.0.0

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 show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @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)

Bumps [got](https://github.com/sindresorhus/got) from 13.0.0 to 14.2.1.
- [Release notes](https://github.com/sindresorhus/got/releases)
- [Commits](sindresorhus/got@v13.0.0...v14.2.1)

---
updated-dependencies:
- dependency-name: got
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Mar 20, 2024
@dependabot dependabot bot requested a review from a team March 20, 2024 19:10
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@djelinek djelinek merged commit 20eb715 into main Mar 20, 2024
11 checks passed
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/got-14.2.1 branch March 20, 2024 19:32
@DavyLandman
Copy link
Contributor

DavyLandman commented Mar 25, 2024

@djelinek I think this should be reverted, as got now requires node 20.0. This is incompatible with the engine requirements for extester (namely node 18).

Strange that @dependabot did not notice this mismatch.

DavyLandman added a commit to SWAT-engineering/vscode-extension-tester that referenced this pull request Apr 11, 2024
@djelinek
Copy link
Collaborator

djelinek commented Apr 16, 2024

Hello @DavyLandman thank you for reporting this back!

Basically you are right and we can revert to get rid of that warning message during npm install.

Few details for the context.

  • ExTester is compatible and can be used also with Node 20+. But at the moment we are supporting and requesting Node 18, because that is a version which is used and supported by VS Code and Electron version which is VS Code built on.
  • for upcoming VS Code release (probably 1.89.x) there should be bump of Electron and also bump of Node from 18 to 20

@DavyLandman
Copy link
Contributor

DavyLandman commented Apr 16, 2024

hi @djelinek I thought that the node 18 dependency was more related to the selenium?

since this library is not running inside of VS Code, it doesn't matter "that much" the be on the same version of node as running inside of VS Code right?

@djelinek
Copy link
Collaborator

Hi, you are right. I have forget to consider also selenium-webdriver NodeJS support policy which is less clear for the current 20+

I will accept your PR, thank you for that 👍🏻

We will be tracking selenium-webdriver and VS Code nodejs support matrix and for now we still have to require NodeJS 18.x

DavyLandman added a commit to SWAT-engineering/vscode-extension-tester that referenced this pull request Apr 17, 2024
@DavyLandman
Copy link
Contributor

@djelinek great. I hope you don't mind the PR, I was just thinking, instead of only leaving issues & comments, I could try to fix something if I see what has to happen.

@djelinek
Copy link
Collaborator

every contribution is very welcomed 🙂 ! Thank you for that

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants