- 
                Notifications
    
You must be signed in to change notification settings  - Fork 275
 
          feat: add static code diagnostic avoid-use-expanded-as-spacer
          #903
        
      Conversation
          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     #903      +/-   ##
==========================================
+ Coverage   86.77%   86.79%   +0.02%     
==========================================
  Files         296      298       +2     
  Lines        6297     6325      +28     
==========================================
+ Hits         5464     5490      +26     
- Misses        833      835       +2     
 Continue to review full report at Codecov. 
  | 
    
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.
What about just SizedBox? It's not the same as Spacer?
| 
           @incendial Spacer semantically correct than SizeBox. SizeBox work like a Container.  | 
    
…as-spacer' into 898-new-rule-avoid-use-expanded-as-spacer
…use-expanded-as-spacer
| 
           Kudos, SonarCloud Quality Gate passed!      | 
    
    
      
        1 similar comment
      
    
  
    | 
           Kudos, SonarCloud Quality Gate passed!      | 
    








…ps://dartcodemetrics.dev/docs/rules/flutter/avoid-use-expanded-as-spacer).
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:
#898
What changes did you make? (Give an overview)
Is there anything you'd like reviewers to focus on?