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

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants