chore: willboosterify this repo#19
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces structural comments to configuration files, updates the @willbooster/wb dependency to version 13.12.13, and renames the configuration variable in oxlint.config.ts. Feedback was provided to revert the variable name oxlintResolvedConfig back to config to adhere to the repository's style guide regarding simplicity and redundancy.
| @@ -1,7 +1,7 @@ | |||
| // wbfy:start oxlint-base | |||
| import config from '@willbooster/oxlint-config'; | |||
| import oxlintResolvedConfig from '@willbooster/oxlint-config'; | |||
There was a problem hiding this comment.
The variable name oxlintResolvedConfig is unnecessarily verbose. Per the repository style guide, simplify code to eliminate redundancy. config is sufficient and consistent with other configuration files in the repository.
| import oxlintResolvedConfig from '@willbooster/oxlint-config'; | |
| import config from '@willbooster/oxlint-config'; |
References
- Simplify code as much as possible to eliminate redundancy. (link)
|
|
||
| // wbfy:start oxlint-export | ||
| export default config; | ||
| export default oxlintResolvedConfig; |
There was a problem hiding this comment.
The variable name oxlintResolvedConfig is unnecessarily verbose. Per the repository style guide, simplify code to eliminate redundancy. config is sufficient and consistent with other configuration files in the repository.
| export default oxlintResolvedConfig; | |
| export default config; |
References
- Simplify code as much as possible to eliminate redundancy. (link)
No description provided.