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

Modify "Española" to "Español" #1607

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Modify "Española" to "Español" #1607

merged 1 commit into from
Feb 28, 2023

Conversation

victorbnl
Copy link
Contributor

@victorbnl victorbnl commented Dec 24, 2022

PR Description:

So the translated name of Spanish is defined as "Española". It's pretty unnatural. At first, I thought it might be a shorthand for something like "la idioma española" : "the Spanish language", which, while still being really strange, would make a little bit of sense. Nevertheless, when looking at the other languages' names, they don't seem to follow this "convention" : French is "Français" and not "Française", Russian is "Ruskij" and not "Ruskaja"...

Tests and Checks

  • I have tested the code!

@castillofrancodamian
Copy link
Contributor

@victorbnl It will have been a mistake by the author who does not speak Spanish like us.

@victorbnl
Copy link
Contributor Author

Yep that's what I came to think eventually

@victorbnl victorbnl closed this by deleting the head repository Dec 29, 2022
@castillofrancodamian
Copy link
Contributor

@victorbnl What happened?

@victorbnl
Copy link
Contributor Author

Well, I deleted my fork of the repository and I thought the pull request was going to stay open, as it used to on GitHub.

@victorbnl victorbnl reopened this Dec 29, 2022
@victorbnl
Copy link
Contributor Author

I think the PR is mergeable again now

@Torxed
Copy link
Member

Torxed commented Feb 28, 2023

@victorbnl It will have been a mistake by the author who does not speak Spanish like us.

It's more an issue of:

@Torxed
Copy link
Member

Torxed commented Feb 28, 2023

Ignore my previous comment, I am a bit tired and thought the two PR's I linked added the a. It simply was a miss to not remove it from the translation file too. Merging!

@Torxed Torxed merged commit e0c20f9 into archlinux:master Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants