Closed Sanlovty closed 2 years ago
@Sanlovty Other than above comment, this look good. Good job! 👏🏻
Merging #60 (a7d5622) into main (d7abcb8) will increase coverage by
0.41%
. The diff coverage is88.88%
.
@@ Coverage Diff @@
## main #60 +/- ##
==========================================
+ Coverage 70.56% 70.97% +0.41%
==========================================
Files 15 15
Lines 564 572 +8
==========================================
+ Hits 398 406 +8
Misses 166 166
Impacted Files | Coverage Δ | |
---|---|---|
lib/src/asset_group.dart | 83.87% <0.00%> (ø) |
|
lib/src/constants.dart | 50.00% <ø> (ø) |
|
lib/src/data/test_template.dart | 80.00% <ø> (ø) |
|
lib/src/dart_class_generator.dart | 91.92% <100.00%> (+0.05%) |
:arrow_up: |
lib/src/data/export_template.dart | 80.00% <100.00%> (+5.00%) |
:arrow_up: |
lib/src/spider_config.dart | 95.45% <100.00%> (+0.71%) |
:arrow_up: |
lib/src/utils.dart | 79.23% <100.00%> (+0.49%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d7abcb8...a7d5622. Read the comment docs.
Thanks @Sanlovty for your contributions. 🙌🏻
Released in v3.2.0
Description of the Change
Added
ignored_rules
property in global scope.Benefits
Now u are able to set your custom set of rules that must be ignored in generated asset-classes. Linter no longer warns.
Possible Drawbacks
Don't exist
Verification Process
Added and passed tests. Linter no longer warns.
Applicable Issues
Closes #55