Closed huwshimi closed 4 months ago
Attention: Patch coverage is 92.30769%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 97.19%. Comparing base (
9787efd
) to head (09799e7
).:exclamation: Current head 09799e7 differs from pull request most recent head cb5c248
Please upload reports for the commit cb5c248 to get more accurate results.
Files | Patch % | Lines |
---|---|---|
src/juju/api.ts | 90.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Note: this will fail CI until https://github.com/juju/js-libjuju/pull/127 is released which needs to be QAed along with this PR.
Done
QA
yarn install
andyarn build
inside the jujulib dir.yarn link
to the jujulib dir you just checked out.Details
https://warthogs.atlassian.net/browse/WD-11707