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: refactor to improve legibility #2432

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blottn
Copy link

@blottn blottn commented May 11, 2024

The use of "not more" with the reverse ordering of the sentence made this difficult to parse. After reversing the order and removing negation, the surrounding paragraphs were updated to match the style.

What issue does this pull request resolve?
None

What changes did you make?
Restructured some paragraphs in the docs

Is there anything that requires more attention while reviewing?
Please validate that this new reordering aligns with the functionality as I have only just begun using this tool.

The use of "not more" with the reverse ordering of the sentence made
this difficult to parse. After reversing the order and removing
negation, the surrounding paragraphs were updated to match the style.
@jasoniangreen
Copy link
Collaborator

Thanks for the contribution @blottn, I can see what you mean about the original wording. I will take a look at this and see, I'll also have to review the actual functionality to ensure it matches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants