Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: Replace sudo-prompt with @expo/sudo-prompt fork #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

kitten
Copy link
Member

@kitten kitten commented Jan 22, 2025

Summary

Related: expo/expo#34364

The sudo-prompt package has been deprecated. See: https://www.npmjs.com/package/sudo-prompt
A fork was maintained at: https://www.npmjs.com/package/@vscode/sudo-prompt
This in turn caused util deprecations, so a new fork with those fixed is at: https://www.npmjs.com/package/@expo/sudo-prompt

This shouldn't have a large impact, but we should maybe look at replacing this package eventually. We didn't need to touch this in a while, so this is purely to solve npm and Node deprecations.

Set of changes

  • Replace sudo-prompt with @vscode/sudo-prompt
  • Remove typings stub

@kitten kitten marked this pull request as draft January 22, 2025 17:10
@kitten kitten marked this pull request as ready for review January 23, 2025 09:52
@kitten kitten changed the title chore: Replace sudo-prompt with @vscode/sudo-prompt fork chore: Replace sudo-prompt with @expo/sudo-prompt fork Jan 31, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant