fix(v4): fix api/advanced/reporters.md error eslint's format - #1006
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2acbf60776
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| 'advanced/api/import-example.md', | ||
| 'api/advanced/import-example.md', | ||
| 'guide/examples/*.md', | ||
| 'reporters-life-cycle.md', |
There was a problem hiding this comment.
Use an ignore glob that reaches the partial
In this flat ESLint setup, a bare reporters-life-cycle.md pattern only matches a file at the config root, while the newly added partial is api/advanced/reporters-life-cycle.md. As a result, CI/pre-commit runs such as eslint . or lint-staged on that staged file will still lint/format the partial, which defeats the purpose of moving the fragile lifecycle list out of reporters.md; use the actual relative path or **/reporters-life-cycle.md instead.
Useful? React with 👍 / 👎.
在提交PR之前,请确保你执行以下操作:
fixes #123)。描述
附加上下文