Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: improve cli output #44

Merged
merged 1 commit into from
Jan 5, 2024
Merged

fix: improve cli output #44

merged 1 commit into from
Jan 5, 2024

Conversation

G-Rath
Copy link
Contributor

@G-Rath G-Rath commented Jan 5, 2024

Before:

❯ bundle exec rake dev:annotate

Chusaku has finished running.
Model files unchanged.

❯ bundle exec rake dev:annotate
Nothing to annotate.
Model files unchanged.

After:

❯ bundle exec rake dev:annotate
Chusaku has finished running.
Model files unchanged.

❯ bundle exec rake dev:annotate
Controller files unchanged.
Model files unchanged.

I've also preserved the newline before the final output line if --verbose is passed.

Resolves #42

nshki
nshki approved these changes Jan 5, 2024
View reviewed changes
Copy link
Owner

@nshki nshki left a comment

Choose a reason for hiding this comment

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

This is lovely. Thank you for this PR!

@nshki nshki merged commit 82d9e22 into nshki:main Jan 5, 2024
6 checks passed
@G-Rath G-Rath deleted the improve-output branch January 21, 2024 19:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@nshki nshki nshki approved these changes

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

Successfully merging this pull request may close these issues.

CLI output can be unclear
2 participants
@G-Rath @nshki