Skip to content

Fixed all grammatical errors in locale_en-US.ini #35053

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Joshdike
Copy link

No description provided.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 11, 2025
@JamesSkemp
Copy link

Based upon casing, I wonder if the period additions for lines 323 and 353 is correct?

1038 and 1039 also read fine to me as they were; the original it's flows better than it is (IMO).

1105 has a url that should probably be URL to match the other instance.

@Joshdike
Copy link
Author

Based upon casing, I wonder if the period additions for lines 323 and 353 is correct?

1038 and 1039 also read fine to me as they were; the original it's flows better than it is (IMO).

1105 has a url that should probably be URL to match the other instance.

The period additions in 323 and 353 doesn't matter but i could remove them.
Same goes for 1038 and 1039.
You are right about 1105. I should make it URL instead of url

@dr-1
Copy link

dr-1 commented Jul 12, 2025

I still see quite a few language issues in the file. The fixes in this PR are great but I would suggest to remove the "all" from the commit message when squashing and merging. Happy to contribute a separate PR after that's done to fix more of the issues.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 12, 2025
@@ -689,7 +689,7 @@ form.name_chars_not_allowed = User name "%s" contains invalid characters.

block.block = Block
block.block.user = Block user
block.block.org = Block user for organization
block.block.org = Block user from organization
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems not right

What's the context?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

context: You are blocking a user from an organization which means restricting access to the organization.
Do you think there's something else you had in mind?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, not a native speaker, but I think I may misunderstand the new sentence.

It reads like "block a user, the user is from an organization"

Old reads like "block a user, then for the organization, the user will have no access"

What do you think?

@@ -3831,7 +3831,7 @@ runners.reset_registration_token_success = Runner registration token reset succe
runs.all_workflows = All Workflows
runs.commit = Commit
runs.scheduled = Scheduled
runs.pushed_by = pushed by
runs.pushed_by = Pushed by
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC it is used in a context, the old i18n keys are not well designed, but I guess we should keep the old text

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, I will change it to the old text. Please confirm everything is okay with line 692 and I will make an update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged. modifies/translation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants