-
Notifications
You must be signed in to change notification settings - Fork 417
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
"These apps are now globally available" is confusing now that --global is a thing #1460
Comments
Hi everyone,
As I don't think the line in itself is wrong (I mean, I only changed a string), I may need some help understanding the tests work so that I can contribute correctly. Any suggestion is welcome. As we are on the topic, @dechamps, do you think it would be better to also change line 358 from |
Thanks and welcome, @NerusSkyhigh!
I would suggest to create a pull request in case there is some local environment releate error. If not, it is still more convenient to deal with the problem directly based on the pull request together.
IMO, it is fine. The |
Hi, thanks for your time @huxuan . For the second change, I'll wait for others' responses and change that if needed. |
We might as well make it consistent while we're at it IMO. |
The phrasing "These apps are now globally available" seems to imply that I asked for a system-wide install using
--global
, but I didn't (and it didn't). This is confusing.Alternatives include:
--global
was passedThe text was updated successfully, but these errors were encountered: