Skip to content
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

feat: Option to hide errors in the cli. #1005

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mlambley
Copy link

@mlambley mlambley commented Apr 27, 2018

Create a non-breaking option which will override inline-errors such that errors can be hidden in the cli altogether.
Useful to reduce clutter in the cli, if another reporter (eg. html or apiary) is reporting the errors.

🚀 Why this change?

To reduce clutter in the CLI when dealing with large API specifications.

📝 Related issues and Pull Requests

#952

✅ What didn't I forget?

  • To write docs
  • To write tests
  • To put Conventional Changelog prefixes in front of all my commits and run npm run lint

@mlambley mlambley changed the title feat: Option for hiding errors in the cli. feat: Option to hide errors in the cli. Apr 27, 2018
@honzajavorek
Copy link
Contributor

Thanks! It might take us a while to go through this. First let's get the Windows build passing - just needs retries, we have some flaky tests 😞

@pksunkara
Copy link
Contributor

@honzajavorek What's the status of this?

@honzajavorek
Copy link
Contributor

@pksunkara I'd be interested in what is exactly the use case, because if we need this, I think there's more stuff wrong with Dredd's output. I've been delaying this because I wanted to revisit #765 until the end of this year.

@honzajavorek honzajavorek added this to the Improve CLI output milestone Jan 12, 2019
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.

4 participants