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

docs: new style guide - Inclusive Language #990

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

BhaswatiRoy
Copy link
Contributor

Fix: asyncapi/website#1248

Added Style Guide for Inclusive Language -

Culturally Inclusive Language
Gender Neutral Language
People First Language
Slang Free Language
Ageism Free Language
Knowledge Assumption Free Language

Continuation: asyncapi/website#1327

Copy link

@CBID2 CBID2 left a comment

Choose a reason for hiding this comment

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

This is a great start. I suggested some ideas that I think can be helpful in explaining the importance of being inclusive

docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
@asyncapi asyncapi deleted a comment from CBID2 Jan 31, 2024
Copy link

@TRohit20 TRohit20 left a comment

Choose a reason for hiding this comment

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

I have suggested some minor changes that you can apply @BhaswatiRoy

Post which, the document to me seems Good to go.

CC @alequetzalli Kindly review from your side and merge if all good.

docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
docs/style-guide/inclusive-language.md Outdated Show resolved Hide resolved
@BhaswatiRoy
Copy link
Contributor Author

I have suggested some minor changes that you can apply @BhaswatiRoy

Post which, the document to me seems Good to go.

CC @alequetzalli Kindly review from your side and merge if all good.

Thank you for the suggestions, I have added the changes

Copy link

@TRohit20 TRohit20 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you @BhaswatiRoy

Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Jul 25, 2024
@thulieblack
Copy link
Member

thulieblack commented Oct 11, 2024

Still waiting for the automation for the docs to be complete before merging this cc @quetzalliwrites

@thulieblack thulieblack removed the stale label Oct 11, 2024
Copy link

@TRohit20 TRohit20 left a comment

Choose a reason for hiding this comment

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

@quetzalliwrites The PR LGTM but has been stale for a while, can you give a final review from your end and merge the PR?

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.

docs: new style guide - Inclusive language
4 participants