-
Notifications
You must be signed in to change notification settings - Fork 633
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
fix(node): don't unnecessarily ask for env access #3132
Conversation
Canary is quite literally not built yet, this should pass when it is finished. |
Quick turnaround! 🚀😳 |
Looks good to me. Thanks for your contribution! But now this is blocked by Deno Deploy compatibility as it doesn't have |
BTW currently we exceptionally skip checking Maybe we should remove that exceptional handling in CLI after this landed. (@bartlomieju What do you think about this?) |
Sure, sounds good to me! |
Co-authored-by: Yoshiya Hinosawa <[email protected]>
Can we re-create this PR in the main repo if it's still needed? |
Closing this PR as it is stale. |
Props to @iuioiua for getting
querySync
landed (denoland/deno#17019).Closes #2446.