Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

Proofread the docstrings for the help output #30

Merged
merged 2 commits into from
Sep 7, 2022
Merged

Proofread the docstrings for the help output #30

merged 2 commits into from
Sep 7, 2022

Conversation

ihatethefrench
Copy link
Member

Fixes #21

Trivernis
Trivernis approved these changes Sep 7, 2022
View reviewed changes
Copy link
Member

@Trivernis Trivernis left a comment

Choose a reason for hiding this comment

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

Not sure who reads the docs but it's fine

@ihatethefrench
Copy link
Member Author

They show up in Clap, and it was bugging me and Oro lol, this is just in general the start of what's going to be a series of improvements (visual and non-visual) for Jade

@ihatethefrench ihatethefrench merged commit 1d5dc7e into main Sep 7, 2022
@Trivernis
Copy link
Member

They show up in Clap, and it was bugging me and Oro lol, this is just in general the start of what's going to be a series of improvements (visual and non-visual) for Jade

Oh lmao
Makes sense then

@ihatethefrench ihatethefrench deleted the help-proofread branch September 17, 2022 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Trivernis Trivernis Trivernis approved these changes

@axtloss axtloss Awaiting requested review from axtloss

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG]: Typos in the CLI
2 participants
@ihatethefrench @Trivernis