refactor: Sort imports with eslint

[lachlancollins]

I personally find imports sorted in a consistent order easier to approach, just like how prettier formats code in a consistent way. The rules here are totally flexible but would mean all packages follow the same structure.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
query	⬜️ Ignored (Inspect)			Jul 16, 2023 4:46am

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 0750784. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

🟥 Failed Commands
nx affected --targets=test:eslint,test:lib,test:types,test:build

✅ Successfully ran 1 target

• nx affected --targets=test:lib --base=8f19f73585439393b0eb6cfb4e794479664a0b80

---

Sent with 💌 from NxCloud.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 0750784:

Sandbox	Source
@tanstack/query-example-react-basic-typescript	Configuration
@tanstack/query-example-solid-basic-typescript	Configuration
@tanstack/query-example-svelte-basic	Configuration
@tanstack/query-example-vue-basic	Configuration

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (ddf6986) 92.29% compared to head (04a2933) 92.29%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5634   +/-   ##
=======================================
  Coverage   92.29%   92.29%           
=======================================
  Files         112      112           
  Lines        4283     4283           
  Branches     1117     1117           
=======================================
  Hits         3953     3953           
  Misses        309      309           
  Partials       21       21           

Impacted Files	Coverage Δ
packages/eslint-plugin-query/src/configs/index.ts	100.00% <ø> (ø)
.../src/rules/exhaustive-deps/exhaustive-deps.rule.ts	100.00% <ø> (ø)
...src/rules/exhaustive-deps/exhaustive-deps.utils.ts	100.00% <ø> (ø)
...-query-object-syntax/prefer-query-object-syntax.ts	96.19% <ø> (ø)
...kages/eslint-plugin-query/src/utils/create-rule.ts	100.00% <ø> (ø)
packages/query-core/src/mutation.ts	94.17% <ø> (ø)
packages/query-core/src/queryCache.ts	100.00% <ø> (ø)
packages/query-core/src/tests/utils.ts	88.57% <ø> (ø)
...s/query-persist-client-core/src/__tests__/utils.ts	89.47% <ø> (ø)
packages/query-sync-storage-persister/src/index.ts	85.00% <ø> (ø)
... and 59 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[TkDodo]

I agree, but it's gonna cause so many conflicts when merging back to alpha 😭

Maybe we should this once v5 is released and we have only one branch to maintain?

[lachlancollins]

I agree, but it's gonna cause so many conflicts when merging back to alpha 😭

Maybe we should this once v5 is released and we have only one branch to maintain?

@TkDodo I was just going to merge main into alpha and only keep the changes to .eslintrc.cjs, then re-run eslint fix. I'm hoping this would actually mean there would be fewer conflicts between v4 and v5!

[TkDodo]

okay, let me just ship the fix for 5538 first. It has two PRs - one for main and one for alpha.