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

Slack guidelines: revise self-promotion CTA language #2001

Merged
merged 3 commits into from
Aug 27, 2023

Conversation

ddbeck
Copy link
Member

@ddbeck ddbeck commented Aug 7, 2023

Not everyone knows what an infomercial looks like. Here, we try to be more specific about what we're talking about.


📚 Documentation preview 📚: https://writethedocs-www--2001.org.readthedocs.build/

Copy link
Contributor

@plaindocs plaindocs left a comment

Choose a reason for hiding this comment

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

I feel like first line and last line say pretty much the same thing, and would delete or consolidate, but don't feel strongly on the matter.

Copy link
Collaborator

@janine-c janine-c left a comment

Choose a reason for hiding this comment

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

Sorry about the delay on this one! Looks great - no one can accuse us of not giving enough detail this way 😄

@ddbeck ddbeck enabled auto-merge August 27, 2023 13:20
@ddbeck ddbeck merged commit 994b42f into writethedocs:main Aug 27, 2023
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.

4 participants