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

Add newlines to prints #273

Merged
merged 2 commits into from
Jan 28, 2025
Merged

Add newlines to prints #273

merged 2 commits into from
Jan 28, 2025

Conversation

AuStien
Copy link
Contributor

@AuStien AuStien commented Jan 28, 2025

Improves readability of error outputs.

Old output:

kustomize-sops $ ksops asd
unable to read in manifest: asdkustomize-sops $

New output:

kustomize-sops $ ./ksops asd
unable to read in manifest: asd
kustomize-sops $

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. kind:enhancement New feature or request labels Jan 28, 2025
devstein
devstein approved these changes Jan 28, 2025
View reviewed changes
Copy link
Collaborator

@devstein devstein left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Helpful. Thanks for contributing @AuStien!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 28, 2025
@devstein devstein merged commit 294de9d into viaduct-ai:master Jan 28, 2025
3 checks passed
@AuStien AuStien deleted the print-newline branch January 28, 2025 18:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@devstein devstein devstein approved these changes

Assignees
No one assigned
Labels
kind:enhancement New feature or request lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AuStien @devstein