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

[AC-2224] [deps]: Update open to v10 #456

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Feb 17, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
open 8.4.2 -> 10.0.3 age adoption passing confidence

Release Notes

sindresorhus/open (open)

v10.0.3

Compare Source

v10.0.2

Compare Source

v10.0.1

Compare Source

  • Add Windows environment variable fallback for some broken systems (#​328) 8e69be4

v10.0.0

Compare Source

Breaking

v9.1.0

Compare Source

v9.0.0

Compare Source

Breaking
  • Require Node.js 14 7f5995e
  • This package is now pure ESM. Please read this.
    • Please don't open issues regarding ESM / CommonJS.
  • open.openApp is now a named import: import {openApp} from 'open'
  • open.apps is now a named import: import {apps} from 'open'
Improvements
  • Add the ability to open default browser and default browser in private mode (#​294) 3b79981

Configuration

📅 Schedule: Branch creation - "every weekend" (UTC), Automerge - At any time (no schedule defined).

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

Rebasing: Whenever PR is behind base branch, 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 has been generated by Mend Renovate. View repository job log here.

AC-2224

@renovate renovate bot requested a review from a team as a code owner February 17, 2024 00:51
@renovate renovate bot requested a review from addisonbeck February 17, 2024 00:51
@bitwarden-bot bitwarden-bot changed the title [deps]: Update open to v10 [AC-2224] [deps]: Update open to v10 Feb 17, 2024
@bitwarden-bot
Copy link

Internal tracking:

@bitwarden-bot
Copy link

bitwarden-bot commented Feb 17, 2024

Logo
Checkmarx One – Scan Summary & Details9b982557-6307-4e0d-bc2e-afc2649201e3

No New Or Fixed Issues Found

Copy link
Contributor Author

renovate bot commented Feb 17, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@addisonbeck
Copy link
Contributor

I can't find a place where this package is used.

I've been using a dependency audit tool called depcheck, and it reports this package as being in use somewhere in the project. BUT I can't find it anywhere. Here are some notes:

  1. There are no import references found for this package doing a typical grep search
  2. The package, open, is a cli tool used for opening urls, files, etc. in non web-apps.
  3. There is a recommended alternative to open for electron on the package README
  4. Directory Connector doesn't open any urls, RIGHT? I might just be missing something here.
  5. The app loads and functions just fine without the package installed:
Screenshot 2024-02-16 at 8 09 25 PM

Seeking a second opinion on this one before merging to remove the package instead of updating it.

Copy link
Member

@eliykat eliykat left a comment

Choose a reason for hiding this comment

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

It's used in cliPlatformUtilsService, however this doesn't seem to be used in directory connector. I'm OK with removing it - in fact, this is not a bad way to work our way through unused dependencies.

It would also be good to delete the dead code so that there's not code in the project that is intentionally missing dependencies. However, that might be a more comprehensive effort as imports seem to cascade through jslib quite easily.

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.

None yet

3 participants