-
Notifications
You must be signed in to change notification settings - Fork 131
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
chore: update error messages #1571
Conversation
🦋 Changeset detectedLatest commit: b2761fe The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Coverage report
Test suite run success735 tests passing in 102 suites. Report generated by 🧪jest coverage report action from b2761fe |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor thing, also I suggest we add a changelog here despite it being a chore. It wouldn't be too surprising if people were matching error messages in their own code, so we should probably let them know that they might see a change.
Co-authored-by: Lorna Jane Mitchell <github@lornajane.net>
What/Why/How?
Updated a couple of error messages to make the format more consistent.
We don't need the extra lines as the exitWithError function already adds them.
Also corrected the wrong url: https://github.com/Redocly/redocly-cli/pull/1571/files#diff-2fb448d11702324cc59d657d5945d59cd325540eb87b71f2e66a5bc01ada834cR730
Reference
Testing
Screenshots (optional)
Check yourself
Security