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: Add support to change files tree in reporter . --summarizer=flat|nested|pkg option #1370

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

Conversation

blackswanny
Copy link

During debug i found there is no way to change how files are represented in report (flat, tree, etc) despite this feature is available in istanbul-lib-report package that generates files nodes tree for report. This allows to pass override of this feature from nyc down to istanbul-lib-report

@michaelhays
Copy link

Is the failed check preventing this from being merged? This option would be really useful!

@blackswanny
Copy link
Author

@coreyfarrell it fails on some particular NodeJS vers , but not all. And idk, why my change would have caused it. It shouldn't be related

@michaelhays
Copy link

@coreyfarrell Any chance we could get this merged? This would be an extremely useful workflow improvement.

@blackswanny Maybe you could try merging master into this branch to see if the tests would work. I wonder if the test failures were just flakes.

@Gu7z
Copy link

Gu7z commented Feb 28, 2024

Is this project still actively maintained? I believe implementing this feature would be incredibly beneficial and enhance the utility of the project significantly. Looking forward to any updates on this.

@SimenB @bcoe

@michaelhays
Copy link

@Gu7z No, there hasn't been a commit here in nearly 3 years. I'd recommend using https://github.com/bcoe/c8

@SimenB
Copy link
Member

SimenB commented Feb 28, 2024

I'm not a maintainer, user or consumer of this project

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