Conversation
|
This only fixes the issues that could be automatically fixed. There's still a handful of things that couldn't be automatically addressed. |
mamhoff
left a comment
There was a problem hiding this comment.
I like this change, and I think it sets us up on a good track. Why not mark it as ready for review?
|
I just bugged @AlistairNorman to get this wrapped when he has time. Would love to see this put to bed once and for all. |
|
I'll definitely work on it but there are 94 offences that can't be automatically corrected |
deaa49d to
5196741
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #6421 +/- ##
=======================================
Coverage 89.66% 89.66%
=======================================
Files 990 990
Lines 20798 20792 -6
=======================================
- Hits 18649 18644 -5
+ Misses 2149 2148 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Just put them into the ignore/to-do file for now. They can be corrected later! |
412bb5f to
c6ddf0c
Compare
68afe7f to
ac6a471
Compare
Co-Authored-By: Jared Norman <jared@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Senem Soy <senem@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd> Co-Authored-By: Noah Silvera <noah@super.gd>
Co-Authored-By: Jared Norman <jared@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Senem Soy <senem@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd> Co-Authored-By: Noah Silvera <noah@super.gd>
These changes have been reviewed individually to ensure these are safe. Co-Authored-By: Jared Norman <jared@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Senem Soy <senem@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd> Co-Authored-By: Noah Silvera <noah@super.gd>
Standard wants the shared logic out of both branches, but the condition mutates the order, so the sequence matters. Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
Co-Authored-By: Jared Norman <jared@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Senem Soy <senem@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd> Co-Authored-By: Noah Silvera <noah@super.gd>
These do need to be or equals so that we don't overwrite these if people have changed them. Co-Authored-By: Jared Norman <jared@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Senem Soy <senem@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd> Co-Authored-By: Noah Silvera <noah@super.gd>
This isn't from Standard. Alistair just likes it better. (So does everyone else.) Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
ac6a471 to
3a29579
Compare
Co-Authored-By: Alistair Norman <alistair@super.gd> Co-Authored-By: Chris Todorov <chris@super.gd> Co-Authored-By: Adam Mueller <adam@super.gd>
3a29579 to
453e8fc
Compare
Summary
This builds off of #6241
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
The following are not always needed: