Skip to content
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

English language minor corrections in diagnostic.rb #16120

Closed
wants to merge 1 commit into from

Conversation

Rezonansowy
Copy link

Minor natural language fixes; sticking with US English convention that is predominantly present in the code.

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew typecheck with your changes locally?
  • Have you successfully run brew tests with your changes locally?

Minor natural language fixes; sticking with US English convention that is predominantly present in the code.
@MikeMcQuaid
Copy link
Member

Sorry, passing on this, we intentionally use en_GB as documented in our Prose Guidelines on docs.brew.sh.

@Rezonansowy
Copy link
Author

@MikeMcQuaid I do not believe that you fully reviewed the commit in my pull request. There were only two changes regarding the American English spelling, and the remaining three changes were to fix blatant English errors that are not specific for US English and are still there. I recall I marked an option “Allowing edits by maintainers”, so you could simply delete those two redundant u letters yourself. Instead, you opted to reject everything, including the changes that were 100% appropriate and had literally nothing to do with en_GB use. Please reconsider.

On a sidenote: Thanks for sharing the Prose Guidelines, but perhaps you should make it really clear it applies to messages printed by the software, as it is not documentation nor “other communications with the Homebrew community”. Right now, it is not clear at all. By the way, I was not aware that UK English is still a thing in a software of modern times.

@MikeMcQuaid
Copy link
Member

the remaining three changes were to fix blatant English errors that are not specific for US English and are still there

I consider all of these to be personal preference and not errors.

By the way, I was not aware that UK English is still a thing in a software of modern times.

It is in some software. This software was created by someone from the UK and the project leader and longest-running maintainer (👋🏻) as well as several of our other maintainers live in the UK.

@github-actions github-actions bot added the outdated PR was locked due to age label Nov 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants