Conversation
|
|
@auvred is attempting to deploy a commit to the Josh Goldberg's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
👋 Hi @auvred, thanks for the pull request! A scan flagged some concerns with it. Could you please take a look? [pr-title-conventional] The PR title has an unknown type: 'WIP'. Known types are: 'build', 'chore', 'ci', 'docs', 'feat', 'fix', 'perf', 'refactor', 'revert', 'style', 'test' This repository asks that pull request titles start with a type in the Conventional Commits format. Doing so helps make the purpose of each pull request clear for humans and machines. To resolve this report, replace the current type with one of those known types, like "feat: Vue.js support". [pr-task-completion] This PR's body is missing
Repositories often provide a set of tasks that pull request authors are expected to complete. Those tasks should be marked as completed with a
|
## PR Checklist - [x] Addresses an existing open issue: - fixes #851 - fixes #852 - fixes #857 - fixes #995 - [x] That issue was marked as [`status: accepting prs`](https://github.com/JoshuaKGoldberg/flint/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22) - [x] Steps in [CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/flint/blob/main/.github/CONTRIBUTING.md) were taken ## Overview Reworks the `/rules` Rules page and sidebar to add an _Incubator Plugins_ section. The tables of all rules are now in their own sub-pages under a new _All Rules_ sidebar group. This adds several hundred rules to tracking by populating the following incubator plugins: * Astro * Next (#847) * Nuxt * React (#846) * SolidJS (#848) * Vitest * Vue (#835, #836) Those incubator plugin lists will need auditing from folks more familiar with the frameworks. Vue's, in particular, I think is probably wrong in many ways. At least: * deprecated: which are syntactic, covered by `@deprecated` checking, neither? * Naming: props vs attributes? * Should there be a "migration" plugin for migrating from Vue 2 to Vue 3? Also applies some cleanups: * Plugin objects' names are now their TitleCase form, not lowercase * Comparisons data is tweaked a bit and has a sorting script alongside it * Renames the `cspell` plugin to `spelling` ❤️🔥
|
This PR is impossible to review all at once, so I'll split it into several smaller PRs |
|
Closing in favor of #2315 |
1 participant
PR Checklist
status: accepting prsOverview