chore: simplified eslint.config #439
Conversation
|
@JoshuaKGoldberg is attempting to deploy a commit to the Brad Garropy's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## eslint #439 +/- ##
==========================================
Coverage ? 100.00%
==========================================
Files ? 165
Lines ? 4706
Branches ? 152
==========================================
Hits ? 4706
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Sentry. |
|
@all-contributors add @JoshuaKGoldberg for review and tool |
|
I've put up a pull request to add @JoshuaKGoldberg! 🎉 |
|
I took all of your suggestions into account, and here's what I ended up with, so clean! https://github.com/bradgarropy/bradgarropy.com/blob/eslint/eslint.config.js Thank you for all of your help, I added you as a contributor! |
|
🙌 awesome! |
Adds @JoshuaKGoldberg as a contributor for review, tool. This was requested by bradgarropy [in this comment](#439 (comment)) --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Co-authored-by: Brad Garropy <[email protected]>
Applies a bunch of simplifications to the file. I've commented them inline. The biggest one is using
ts(eslint).configto get types and flatten arrays.Please forgive me if you already know a bunch of the comments 😅. I figured they'd be useful in case anybody is reading this.
I'm also not sure I'm not removing some rules that actually are intentional. Definitely not suggesting this should be 100% trusted!
💖