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

Translate bot_cheatsheet.pt.Rmd to Portuguese #684

Merged
merged 4 commits into from
Dec 14, 2023
Merged

Conversation

maelle
Copy link
Member

@maelle maelle commented Aug 24, 2023

No description provided.

@pedropark99
Copy link
Contributor

Review 1 at #687.

@maelle
Copy link
Member Author

maelle commented Oct 23, 2023

@pedropark99 @msperlin In the Spanish translation we've made sure to use gender-neutral phrasing. Is "um editor" for instance gender-neutral? If not would you be able to rephrase? Thanks so much for your work and patience.

@msperlin
Copy link
Contributor

Hi @maelle,

Is "um editor" for instance gender-neutral?

No, its not. In portuguese, we usually use "um (a) editor (a)". @pedropark99, since you are the main author, can you please change it directly?

@msperlin
Copy link
Contributor

We might also need to check the other files for this...

@pedropark99
Copy link
Contributor

Hey! I agree with @msperlin! We can reach a more "neutral gender" level by using "um (a)". I will do the change for us ✌️

@pedropark99
Copy link
Contributor

Fixes at #719

@pedropark99
Copy link
Contributor

I understand @maelle that we need to merge #719. After we merge #719, then, we are ready to merge this PR, because we did review 1 and 2 already.

Just as a remainder, the #719 contains the fix that @msperlin suggested in the above comments, to add a "(a)" suffix for a more "neutral gender" language.

@yabellini yabellini merged commit 67fa52d into main Dec 14, 2023
2 checks passed
@yabellini yabellini deleted the bot_cheatsheet.pt branch December 14, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants