| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Some things still don't work though... I couldn't get gcov to work, and grep -o doesn't work with +/* in the regex for me
|
Just for reference, the original scripts are still available at https://github.com/addaleax/node-core-coverage |
Sorry, something went wrong.
|
I can't quite decode your title "translate info into WIP scripts". You are trying to create scripts that can be run outside the build environment ? |
Sorry, something went wrong.
@mhdawson Yes but ideally both? Coverage is... a lot less useful unless you can also run it locally... |
Sorry, something went wrong.
|
Agreed, my goal is to get us to "make coverage" as the way to build/run coverage and @CurryKitten is working to pull more of the pieces needed for that into the core. I'll try to catch up with him to see how far away we are on that front. |
Sorry, something went wrong.
|
As @mhdawson mentioned, I'm working on some additional changes which will effectively give you a coverage result from running make so a developer will be able to see what impact a code change/fix/new test has on the overall coverage result. Next action is to add Javascript coverage into the build. |
Sorry, something went wrong.
|
@CurryKitten Is your work on a public repo somewhere? |
Sorry, something went wrong.
|
@Fishrock123 I'll push my code into my fork (https://github.com/CurryKitten/node) before I make a PR. Should have something ready next week, but I'll let you know |
Sorry, something went wrong.
|
Now that we have make coverage, should this be closed? |
Sorry, something went wrong.
Seems like it, if not it can be reopened. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Some things still don't work though...
I couldn't get gcov to work, and grep -o doesn't work with +/* in the
regex for me
cc @addaleax & @mhdawson