Skip to content
This repository was archived by the owner on Jul 16, 2023. It is now read-only.

feat: add static code diagnostics format-comment. #622

Merged
merged 34 commits into from
Feb 21, 2022

Conversation

vlkonoshenko
Copy link
Member

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:

#566

@vlkonoshenko vlkonoshenko self-assigned this Dec 27, 2021
@vlkonoshenko vlkonoshenko changed the title 566 capitalize comment feat: 566 capitalize comment Dec 27, 2021
@github-actions
Copy link

github-actions bot commented Dec 27, 2021

Dart Code Metrics analyze report of dart_code_metrics. ✅

Summary

  • Scanned folders: bin, example, lib, test

  • Total scanned files: 398

  • Total lines of source code: 7049

  • Total classes: 265

  • Average Cyclomatic Number per line of code: 0.34

  • Average Source Lines of Code per method: 6

  • Total tech debt: 1142.0 hours

  • Found issues: 6 ⚠

@incendial incendial added this to the 4.9.0 milestone Jan 3, 2022
@incendial incendial added the type: enhancement New feature or request label Jan 3, 2022
@incendial incendial changed the title feat: 566 capitalize comment feat: add format-comment rule Jan 3, 2022
@vlkonoshenko vlkonoshenko requested a review from incendial January 4, 2022 13:40
Konoshenko Vlad added 2 commits January 4, 2022 23:56
@codecov
Copy link

codecov bot commented Jan 4, 2022

Codecov Report

Merging #622 (e6e195f) into master (3e6b525) will increase coverage by 0.05%.
The diff coverage is 91.66%.

❗ Current head e6e195f differs from pull request most recent head 11c66e0. Consider uploading reports for the commit 11c66e0 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #622      +/-   ##
==========================================
+ Coverage   86.57%   86.63%   +0.05%     
==========================================
  Files         257      261       +4     
  Lines        5490     5550      +60     
==========================================
+ Hits         4753     4808      +55     
- Misses        737      742       +5     
Impacted Files Coverage Δ
...c/analyzers/lint_analyzer/rules/rules_factory.dart 94.80% <0.00%> (-1.25%) ⬇️
...rules_list/format_comment/format_comment_rule.dart 87.09% <87.09%> (ø)
...rules_list/format_comment/models/comment_info.dart 100.00% <100.00%> (ø)
...rules_list/format_comment/models/comment_type.dart 100.00% <100.00%> (ø)
...lyzer/rules/rules_list/format_comment/visitor.dart 100.00% <100.00%> (ø)

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 3e6b525...11c66e0. Read the comment docs.

@vlkonoshenko vlkonoshenko requested a review from incendial January 8, 2022 14:39
@github-actions
Copy link

github-actions bot commented Jan 10, 2022

Dart Code Metrics unused files report of dart_code_metrics. ✅

Summary

  • Scanned package folders: bin, example, lib
  • No unused files found! ✅

@dkrutskikh dkrutskikh modified the milestones: 4.9.0, 4.10.0 Jan 10, 2022
@dkrutskikh dkrutskikh removed this from the 4.10.0 milestone Jan 31, 2022
@vlkonoshenko vlkonoshenko changed the title feat: add format-comment rule feat: add static code diagnostics format-single-line-comment. Feb 8, 2022
@vlkonoshenko vlkonoshenko changed the title feat: add static code diagnostics format-single-line-comment. feat: add static code diagnostics format-comment. Feb 21, 2022
@dkrutskikh dkrutskikh merged commit 4a2f6c2 into master Feb 21, 2022
@dkrutskikh dkrutskikh deleted the 566_capitalize_comment branch February 21, 2022 17:48
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
area-rules type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants