simolus3 / web3dart

Ethereum library, written in Dart.
MIT License
441 stars 270 forks source link

note about analyzer warnings in generated files #223

Closed vinicentus closed 2 years ago

vinicentus commented 2 years ago

Add note about suppressing analyzer warnings for generated files.

Closes #222

codecov-commenter commented 2 years ago

Codecov Report

Merging #223 (48809b0) into master (a63b113) will decrease coverage by 2.51%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #223      +/-   ##
==========================================
- Coverage   71.41%   68.89%   -2.52%     
==========================================
  Files          53       53              
  Lines        2183     2183              
==========================================
- Hits         1559     1504      -55     
- Misses        624      679      +55     
Impacted Files Coverage Δ
lib/src/contracts/generated_contract.dart 0.00% <0.00%> (-42.86%) :arrow_down:
test/infura_integration_test.dart 53.33% <0.00%> (-40.01%) :arrow_down:
lib/src/generated/erc20.g.dart 0.00% <0.00%> (-15.52%) :arrow_down:
lib/src/core/client.dart 5.44% <0.00%> (-12.93%) :arrow_down:
lib/src/contracts/deployed_contract.dart 0.00% <0.00%> (-10.00%) :arrow_down:
lib/src/contracts/abi/abi.dart 88.88% <0.00%> (-5.56%) :arrow_down:
lib/src/credentials/address.dart 74.35% <0.00%> (-5.13%) :arrow_down:
lib/src/core/filters.dart 17.94% <0.00%> (-3.85%) :arrow_down:

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 a63b113...48809b0. Read the comment docs.

simolus3 commented 2 years ago

I don't know how this could have possibly impacted coverage, it's probably something with a newer Dart version chosen for this CI run...

Thanks for adding the documentation :+1: