Skip to content

Drive towards full coverage  #431

Open

Description

@mhdawson

Next steps
1- Blog post to point to the issues we have open to encourage external help.

Activity

  1. added this to the Milestone 11 milestone on Nov 12, 2021
  2. mhdawson commented on Nov 26, 2021

    @mhdawson
    MemberAuthor

    @JckXia has a draft for the blog post, will send to rest of team members through email to discussion/review.

  3. mhdawson commented on Dec 10, 2021

    @mhdawson
    MemberAuthor
  4. KevinEady commented on Apr 12, 2024

    @KevinEady
    Contributor

    @KevinEady to investigate adding a coverage badge to the root README.md.

  5. mhdawson commented on May 3, 2024

    @mhdawson
    MemberAuthor

    Badge is now on README.md

  6. KevinEady commented on May 17, 2024

    @KevinEady
    Contributor

    Next step: Create a "condensed report" which analyzes why some coverage paths are not taken and present this to team for triage.

  7. KevinEady commented on Sep 6, 2024

    @KevinEady
    Contributor
    • Investigate why coverages is lower than expected: is it possible that coverages are being overwritten? Coverages are stored on the filesystem when the process runs (ends?). Could it be the case that since we use subprocesses, a subprocess' test results get overridden when the parent process' finishes?
    • Improve coverage report with experimental runs node-addon-api#1525
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions