fix(lint): assign anonymous default exports to named consts - #135
fix(lint): assign anonymous default exports to named consts#135Whiznificent wants to merge 1 commit into
Conversation
Resolves import/no-anonymous-default-export warnings from next/core-web-vitals in lib/actions/cached-api.js and lib/utils/cloudinaryUpload.js. The default exports were unused across the codebase; converting them to named consts before exporting silences the lint warnings without changing runtime behavior.
|
Someone is attempting to deploy a commit to the Deen Bridge Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: π Files selected for processing (2)
WalkthroughThe change names the default-export objects in ChangesNamed module exports
Estimated code review effort: 1 (Trivial) | ~2 minutes π₯ Pre-merge checks | β 5β Passed checks (5 passed)
β¨ Finishing Touchesπ§ͺ Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. π§ ESLint
ESLint install timed out. The project may have too many dependencies for the sandbox. Comment |
|
π Friendly ping for a maintainer. This PR silences two `import/no-anonymous-default-export` warnings from `next/core-web-vitals` with a minimal, mechanical change (anonymous `export default { ... }` β `const foo = { ... }; export default foo;`). Both default exports were verified unused across the repo before the change. Local verification on the pushed commit:
CI status note: the `Lint and Build` Actions run is sitting at `action_required` because this is a fork PR β a maintainer needs to click Approve and run before the workflow can access secrets. The Vercel preview check is similarly blocked on fork authorization. Once approved, the lint-and-build job should go green on the first run since the local results already match the CI commands exactly. |
|
@Whiznificent this PR has merge conflicts with the |
Summary
Resolves `import/no-anonymous-default-export` warnings from `next/core-web-vitals` ESLint config.
Changes
Verification
The default exports were verified unused across the codebase via ripgrep; the change is purely a lint compliance fix with no runtime behavior impact.
Closes the lint failure surfaced in #144.
Summary by CodeRabbit