[scripts]: add message before running sudo in script - #1117
Conversation
|
@saehejkang My bad, I was looking at PRs top down today and didn't see that you filed yours before the other. I've dropped a suggestion for a EUID check so let's see if the other contributor is cool with the change and merge that one. Did you see my comment on the |
No worries, as it was a super simple change. It might be time to start transitioning to helping out new contributors on "good first issues", instead of working on them myself.
Yes, I saw the comments. Just afk atm and I'll respond to those here soon. |
I think so. The registry one down in containerization is a good choice. There's no way we'd get to it this soon, it ties in nicely with the resource work, and it seems really useful. |
|
Closing in favor of #1118 |
Type of Change
Motivation and Context
Closes #1111
Testing