-
Notifications
You must be signed in to change notification settings - Fork 213
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
Update README.adoc #925
Update README.adoc #925
Conversation
nathansit
commented
Mar 14, 2024
- Just fixed some typos
- Adjusted some wording
- Feel free to accept whichever parts make sense (if any)
@@ -70,7 +69,7 @@ $ brew install mvndaemon/homebrew-mvnd/mvnd | |||
|
|||
We're looking for contribution to support https://www.macports.org[MacPorts], | |||
https://community.chocolatey.org/packages/mvndaemon/[Chocolatey], https://scoop.sh/[Scoop] or | |||
https://github.com/joschi/asdf-mvnd#install[asdf]. If you're fancy helping us... | |||
https://github.com/joschi/asdf-mvnd#install[asdf]. If you fancy helping us... |
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.
-1 on this change. The original was correct IMO.
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.
I'm not a native speaker, but IIUC the original is incorrect, and the suggestion is correct. See also examples e.g. here.
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.
I can see it being written as "If you are fancy, help us" or something along those lines. But I haven't heard that turn of phrase too often either.
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.
Thanks for pointing to an evidence, gentlemen. I did not know "fancy" was also a verb.
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.
The other two changes LGTM :)
f5ce3eb
to
c3831bb
Compare
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.
Thanks @nathansit!