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): update dependency glob to v11 #2327

Closed

Conversation

renovate-bot
Copy link
Contributor

@renovate-bot renovate-bot commented Jul 12, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
glob ^10.3.10 -> ^11.0.0 age adoption passing confidence

Release Notes

isaacs/node-glob (glob)

v11.0.0

Compare Source

v10.4.5

Compare Source

v10.4.4

Compare Source

v10.4.3

Compare Source

v10.4.2

Compare Source


Configuration

📅 Schedule: Branch creation - "before 3am on Friday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate-bot renovate-bot requested a review from a team as a code owner July 12, 2024 01:44
@forking-renovate forking-renovate bot added the dependencies Pull requests that update a dependency file label Jul 12, 2024
Copy link

codecov bot commented Jul 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.14%. Comparing base (dfb2dff) to head (549348b).
Report is 189 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2327      +/-   ##
==========================================
- Coverage   90.97%   90.14%   -0.83%     
==========================================
  Files         146      143       -3     
  Lines        7492     7024     -468     
  Branches     1502     1484      -18     
==========================================
- Hits         6816     6332     -484     
- Misses        676      692      +16     

see 65 files with indirect coverage changes

@blumamir
Copy link
Member

The new version dropped support for node < 20.

If we accept if, the following warning will be emitted with npm ci and node 18 for example:

npm ci     
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'glob@11.0.0',
npm WARN EBADENGINE   required: { node: '20 || >=22' },
npm WARN EBADENGINE   current: { node: 'v18.17.1', npm: '10.2.4' }
npm WARN EBADENGINE }

Since the new version change is only dropping old node versions support, and since this dependency is only used for the internal update-otel-deps.js script. I think we should not accept this new version until we drop support for old node runtimes.

Closing the PR for now

@blumamir blumamir closed this Jul 23, 2024
Copy link

Renovate Ignore Notification

Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future 11.x releases. But if you manually upgrade to 11.x then Renovate will re-enable minor and patch updates automatically.

If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.

@renovate-bot renovate-bot deleted the renovate/glob-11.x branch July 23, 2024 07:02
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants