dart-lang
Merge `coverage` package
Merging package:coverage into this repo.
This is the "raw" output of using https://github.com/mosuem/merger, specifically running
dart run bin/merger.dart --input-name coverage --branch-name master --input-path ~/projects/coverage/ --target-path ~/projects/tools/ --git-filter-repo /home/mosum/tools/git-filter-repo
on my local machine, and some small cleanups afterwards.
- [x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:
- See our contributor guide for general expectations for PRs.
- Larger or significant changes should be discussed in an issue before creating a PR.
- Contributions to our repos should follow the Dart style guide and use
dart format. - Most changes should add an entry to the changelog and may need to rev the pubspec package version.
- Changes to packages require corresponding tests.
Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.
Pull Request Test Coverage Report for Build 10367625157
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
- For more information on this, see Tracking coverage changes with pull request builds.
- To avoid this issue with future PRs, see these Recommended CI Configurations.
- For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
Details
- 571 of 609 (93.76%) changed or added relevant lines in 7 files are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage decreased (-6.2%) to 93.76%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
|---|---|---|---|
| pkgs/coverage/lib/src/chrome.dart | 60 | 61 | 98.36% |
| pkgs/coverage/lib/src/util.dart | 55 | 56 | 98.21% |
| pkgs/coverage/lib/src/formatter.dart | 64 | 66 | 96.97% |
| pkgs/coverage/lib/src/resolver.dart | 78 | 82 | 95.12% |
| pkgs/coverage/lib/src/run_and_collect.dart | 9 | 13 | 69.23% |
| pkgs/coverage/lib/src/collect.dart | 177 | 190 | 93.16% |
| pkgs/coverage/lib/src/hitmap.dart | 128 | 141 | 90.78% |
| <!-- | Total: | 571 | 609 |
| Totals | |
|---|---|
| Change from base Build 10024145560: | -6.2% |
| Covered Lines: | 571 |
| Relevant Lines: | 609 |
💛 - Coveralls
Can you rerun your merger tool? This is missing https://github.com/dart-lang/coverage/pull/502 (and https://github.com/dart-lang/coverage/pull/503 but I see you did a similar change yourself).
