-
Notifications
You must be signed in to change notification settings - Fork 12k
karma-coverage-istanbul-reporter author requires payment for support and issue reporting #17757
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
Comments
I forgot to use |
Updated the title to reflect the fact that the plugin is actually working if used as documented. |
FYI: There's a draft MR for switching to karma-coverage: #17901 |
I'm also facing this Unknown% issue when upgraded from Angular 6 to 9. Are we going to change karma-coverage-Istanbul-reporter to karma-coverage? |
Hey there, I just spotted this issue while looking for something else and thought I should offer an explanation. I've been writing and maintaining open source in my free time for the last 5 years and never earned a penny from it until earlier this year, it's extremely exhausting working on something in your spare time for free, especially when you know that the majority of your users are using it for profit. Charging for support on some of my repos was an experiment to see if I could make maintenance less of a burden, but I can now see that it wasn't a good idea, and I've taken it off in response to this issue as I think it causes more harm than good. I've got some better ideas around open source monetisation that are a bit less abrasive and work better to satisfy the needs of the community. Thanks for raising this issue and I really do appreciate the feedback ✌️ |
This commit switches coverage tooling from karma-coverage-istanbul-reporter to karma-coverage since it's better supported. Closes angular#17757
This commit switches coverage tooling from karma-coverage-istanbul-reporter to karma-coverage since it's better supported. Closes #17757
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Uh oh!
There was an error while loading. Please reload this page.
🐞 Bug report
Command (mark with an
x
)Is this a regression?
I don't know but I think it must have worked.
Description
I have installed a starter through
ng new
leaved everything the default and tried to create a coverage report by enablingkarma-coverage-istanbul-reporter
withreporters: ['progress', 'coverage-istanbul', 'kjhtml'],
. This is how the author describes the installation on his repo. Sadly this does not work and just printsI tried to fill an issue but this author REALLY WANTS $100/month for that!!! This is in acceptable for an OSS project despite the fact that it is coupled to Angular. This makes me angry because new developer will see this and will get a wrong definition of what OSS is and maybe follow this behaviour. Sure, include a sponsor button and say that you don't want to work on it without receiving a SMALL amount of money but then add a "help wanted" badge and ask for help or use issuehunt. I would love to help to solve this issue but nope, my issue gets closed and locked (like many others).
I suggest removing that reporter from the starter and let the author know that this isn't a behaviour Angular supports.
🔬 Minimal Reproduction
Fresh starter with coverage enabled.
🔥 Exception or Error
🌍 Your Environment
Anything else relevant?
I don't know which coverage reporter I should use now...
The text was updated successfully, but these errors were encountered: