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

Remove gender references #151

Merged
merged 13 commits into from
Mar 6, 2024

Conversation

RCheesley
Copy link
Contributor

Hi folks!

👋 Intro

This PR is based on #147 (making it easier to find) and fixes #145 - it removes gendered references which are in the feminine, in favour of using gender-neutral they/them/their. I did not find any in the masculine (he/his/him).

🤔 What's it doing?

I've searched for instances of she or her, and replaced with the appropriate gender neutral term. In some cases a small re-write of the sentence is needed for grammatical reasons, and if I spotted an obvious typo at the same place I fixed it.

🛟 Considerations

In discussion with @andreslucena I understand Decidim prefers not to use contractions, so where I spotted them as I was working on a sentence, I fixed them (eg from don't to do not) to save fixing them in a later PR - figured it makes sense to fix as I saw them.

It might also be worth adding some guidance to writers to not use those gendered terms going forward.

@RCheesley RCheesley mentioned this pull request Dec 26, 2023
@andreslucena
Copy link
Member

@RCheesley sorry again for the delay with this PR. Can you merge with develop branch? I want to have a better diff.

If that's too much of a hassle, you can create a new PR with only the relevant changes for this PR (i.e. cherry-picking fde6579 and 6f7caed)

Thanks!

@RCheesley
Copy link
Contributor Author

@andreslucena hopefully just done that!

@RCheesley
Copy link
Contributor Author

@andreslucena anything else I need to do here? Looks like things are failing :(

Copy link
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

I need to actually allocate time to check out what's happening with the netlify preview. At the moment we can ignore it, as I've checked the PR locally and everything works perfect. Thanks for the PR and sorry for the delay :/

@andreslucena
Copy link
Member

Merging with the netlify preview integration broken, as it actually compiles. We need to fix that in another PR.

@andreslucena andreslucena merged commit 5e67ad3 into decidim:develop Mar 6, 2024
1 of 5 checks passed
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.

Proposal: Remove gendered references in the docs
2 participants