-
Notifications
You must be signed in to change notification settings - Fork 272
Conversation
@roman-petrov could you please take a look too? Maybe I missed some cases. |
Dart Code Metrics unused files report of dart_code_metrics. ✅Summary
|
Dart Code Metrics analyze report of dart_code_metrics. ✅Summary
|
Codecov Report
@@ Coverage Diff @@
## master #686 +/- ##
==========================================
+ Coverage 86.30% 86.33% +0.02%
==========================================
Files 253 255 +2
Lines 5412 5431 +19
==========================================
+ Hits 4671 4689 +18
- Misses 741 742 +1
Continue to review full report at Codecov.
|
@incendial this looks promising! I am a bit ill and surely will take closer look in a couple of days. |
Sure, get well soon! |
location: nodeLocation( | ||
node: invocation, | ||
source: source, | ||
withCommentOrMetadata: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@incendial please set as false
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[ ] Bug fix
[X] New rule
[ ] Changes an existing rule
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain: