-
Notifications
You must be signed in to change notification settings - Fork 281
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
Import sudo-prompt module into our repo #7398
Conversation
The upstream repo has been archived in 2021 and no longer accepts PRs. This import is in preparation to update the bundled AppleScript applet to include aarch64 support, so it no longer requires Rosetta on M1+ machines. Signed-off-by: Jan Dubois <[email protected]>
Signed-off-by: Jan Dubois <[email protected]>
092e16c
to
7f6071f
Compare
7f6071f
to
9a2769f
Compare
Signed-off-by: Jan Dubois <[email protected]>
Signed-off-by: Jan Dubois <[email protected]>
ac510b0
to
e4e8848
Compare
Sorry for asking prematurely to review this PR, but I think it is now ready.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to see that linter fixes were in separate commits, so we have a view of what the upstream code looked like.
It would have been nice to import the upstream repo with commits (like we did for the src/go/
merge), instead of as a snapshot, but I'm not sure you feel like doing the work…
(Requesting just the jest ignore bit; everything is acceptable.)
e4e8848
to
b287119
Compare
Signed-off-by: Jan Dubois <[email protected]>
b287119
to
38ab7ba
Compare
The upstream repo has been archived in 2021 and no longer accepts PRs.
This import is in preparation to update the bundled AppleScript applet to include aarch64 support (#7336), so it no longer requires Rosetta on M1+ machines.