Skip to content

Resolve typos in option descriptions#4521

Closed
Jwaegebaert wants to merge 2 commits intopnp:mainfrom
Jwaegebaert:typos
Closed

Resolve typos in option descriptions#4521
Jwaegebaert wants to merge 2 commits intopnp:mainfrom
Jwaegebaert:typos

Conversation

@Jwaegebaert
Copy link
Copy Markdown
Contributor

@Jwaegebaert Jwaegebaert commented Feb 17, 2023

No related issue. Found some typos in the docs for commands I was checking out. This PR resolves them.

Comment thread docs/docs/cmd/spo/tenant/tenant-settings-set.md
@milanholemans milanholemans marked this pull request as draft February 19, 2023 21:19
@Jwaegebaert Jwaegebaert marked this pull request as ready for review February 23, 2023 15:12
@milanholemans milanholemans marked this pull request as draft February 24, 2023 12:19
@milanholemans
Copy link
Copy Markdown
Contributor

milanholemans commented Feb 24, 2023

Nevertheless, could you fix that the default options are on the bottom of the section?

@pnp pnp deleted a comment from milanholemans Feb 24, 2023
@pnp pnp deleted a comment from milanholemans Feb 24, 2023
@Jwaegebaert Jwaegebaert marked this pull request as ready for review March 2, 2023 15:35
@Jwaegebaert
Copy link
Copy Markdown
Contributor Author

Nevertheless, could you fix that the default options are on the bottom of the section?

Done 😄

Copy link
Copy Markdown
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

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

@Adam-it Adam-it self-assigned this Mar 3, 2023
@Adam-it
Copy link
Copy Markdown
Member

Adam-it commented Mar 3, 2023

ready to merge 🚀

@Adam-it
Copy link
Copy Markdown
Member

Adam-it commented Mar 3, 2023

merged manually. Awesome work 👍. You rock 🤩

@Adam-it Adam-it closed this Mar 3, 2023
@Jwaegebaert Jwaegebaert deleted the typos branch March 11, 2023 19:44
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.

3 participants