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

Update wording #9811

Merged
merged 3 commits into from Mar 29, 2023
Merged

Update wording #9811

merged 3 commits into from Mar 29, 2023

Conversation

svx
Copy link
Contributor

@svx svx commented Mar 29, 2023

What does this PR do?

This PR adjusts the wording under /contributing.

  • Remove some weasel words
  • Add commas
  • For links, replace “here” with something more informative
  • Make docs more inclusive by removing “him” (be gender-neutral)
  • Remove non-needed “white spaces”
  • Remove old information about Traefik Pilot (related to data collection)
  • Adjust wording of “Docker Compose” to official name

Motivation

Improving the docs on step at a time

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

@ldez ldez changed the base branch from v2.9 to v2.10 March 29, 2023 07:19
@ldez ldez added kind/enhancement a new or improved feature. area/documentation and removed status/0-needs-triage labels Mar 29, 2023
@ldez ldez modified the milestones: 2.9, 2.10 Mar 29, 2023
Copy link
Member

@rtribotte rtribotte left a comment

Choose a reason for hiding this comment

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

LGTM 👌

Copy link
Member

@tomMoulard tomMoulard left a comment

Choose a reason for hiding this comment

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

LGTM 👌

@traefiker traefiker merged commit 7fc07c3 into traefik:v2.10 Mar 29, 2023
9 checks passed
@ldez ldez changed the title docs: update wording Update wording Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants