Skip to content

Conversation

@rich-ayr
Copy link
Contributor

@rich-ayr rich-ayr commented Apr 21, 2025

@rich-ayr
Copy link
Contributor Author

@Myshor kindly pinging, need to supersede other patch.

@Myshor
Copy link
Collaborator

Myshor commented Apr 21, 2025

Closed other one but this one I won't accept as it adds 115 lines and removes 162. Which means it's not small change easy to review.

Calling @alexanderby for that.

@Myshor Myshor requested a review from alexanderby April 21, 2025 18:20
@rich-ayr
Copy link
Contributor Author

rich-ayr commented Apr 23, 2025

Understandable, I think it overall adds to the clarity of the document. I've only rewritten a lot of passages, not changed their meaning. Also added a small note to add a screenshot ;)

PS: I would recommend looking at the rendered versions to compare.

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.

2 participants