Merge ~mertkirpici/juju-lint:topic/coverage into juju-lint:master
Proposed by
Mert Kirpici
Status: | Merged |
---|---|
Approved by: | Martin Kalcok |
Approved revision: | fca2a52d8ef38b12e662f7dadd131d9448d0b951 |
Merged at revision: | ce5d33a2b39e4a133eb6abbabc5105c1c9e0ea55 |
Proposed branch: | ~mertkirpici/juju-lint:topic/coverage |
Merge into: | juju-lint:master |
Diff against target: |
83 lines (+24/-3) 3 files modified
pyproject.toml (+15/-0) tests/unit/test_config.py (+7/-2) tox.ini (+2/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sudeep Bhandari | Approve | ||
🤖 prod-jenkaas-bootstack | continuous-integration | Approve | |
Ramesh Sattaru (community) | Approve | ||
Robert Gildein | Approve | ||
Review via email: mp+436243@code.launchpad.net |
Commit message
Require full unit test coverage
Description of the change
This was missed during the snap template update, we expect %100 test coverage in this project.
This also revealed a sneaky bug in the process. When the jujulint.
To post a comment you must log in.
This merge proposal is being monitored by mergebot. Change the status to Approved to merge.