Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(eslint-plugin): no void queryFn #8925

Merged
merged 15 commits into from
Apr 7, 2025
Merged

feat(eslint-plugin): no void queryFn #8925

merged 15 commits into from
Apr 7, 2025

Conversation

Newbie012
Copy link
Collaborator

@Newbie012 Newbie012 commented Apr 1, 2025

read discussion first - #8927

@github-actions github-actions bot added documentation Improvements or additions to documentation package: eslint-plugin-query labels Apr 1, 2025
Copy link

nx-cloud bot commented Apr 1, 2025

View your CI Pipeline Execution ↗ for commit 9ee8ded.

Command Status Duration Result
nx affected --targets=test:sherif,test:knip,tes... ✅ Succeeded 1m 44s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 4s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-07 08:48:26 UTC

Copy link

pkg-pr-new bot commented Apr 1, 2025

More templates

@tanstack/angular-query-devtools-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-devtools-experimental@8925

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@8925

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@8925

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@8925

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@8925

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@8925

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@8925

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@8925

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@8925

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@8925

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@8925

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@8925

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@8925

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@8925

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@8925

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@8925

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@8925

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@8925

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@8925

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@8925

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@8925

commit: 9ee8ded

Copy link

github-actions bot commented Apr 1, 2025

Sizes for commit 9ee8ded:

Branch Bundle Size
Main
This PR

@Newbie012 Newbie012 marked this pull request as ready for review April 1, 2025 14:30
@Newbie012 Newbie012 changed the title feat(eslint-plugin): no void queryFn [DO NOT MERGE] feat(eslint-plugin): no void queryFn Apr 1, 2025
Copy link

codecov bot commented Apr 7, 2025

Codecov Report

Attention: Patch coverage is 93.93939% with 2 lines in your changes missing coverage. Please review.

Project coverage is 44.30%. Comparing base (644e945) to head (9ee8ded).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #8925      +/-   ##
==========================================
+ Coverage   44.09%   44.30%   +0.20%     
==========================================
  Files         201      202       +1     
  Lines        8025     8058      +33     
  Branches     1767     1776       +9     
==========================================
+ Hits         3539     3570      +31     
- Misses       4058     4060       +2     
  Partials      428      428              
Components Coverage Δ
@tanstack/angular-query-devtools-experimental ∅ <ø> (∅)
@tanstack/angular-query-experimental 88.08% <ø> (ø)
@tanstack/eslint-plugin-query 88.19% <93.93%> (+0.37%) ⬆️
@tanstack/query-async-storage-persister 43.85% <ø> (ø)
@tanstack/query-broadcast-client-experimental ∅ <ø> (∅)
@tanstack/query-codemods 0.00% <ø> (ø)
@tanstack/query-core 94.77% <ø> (ø)
@tanstack/query-devtools 3.61% <ø> (ø)
@tanstack/query-persist-client-core 57.73% <ø> (ø)
@tanstack/query-sync-storage-persister 84.61% <ø> (ø)
@tanstack/react-query 95.42% <ø> (ø)
@tanstack/react-query-devtools 10.00% <ø> (ø)
@tanstack/react-query-next-experimental ∅ <ø> (∅)
@tanstack/react-query-persist-client 100.00% <ø> (ø)
@tanstack/solid-query 78.48% <ø> (ø)
@tanstack/solid-query-devtools ∅ <ø> (∅)
@tanstack/solid-query-persist-client 100.00% <ø> (ø)
@tanstack/svelte-query 88.15% <ø> (ø)
@tanstack/svelte-query-devtools ∅ <ø> (∅)
@tanstack/svelte-query-persist-client 100.00% <ø> (ø)
@tanstack/vue-query 71.01% <ø> (ø)
@tanstack/vue-query-devtools ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@TkDodo TkDodo merged commit 993e838 into main Apr 7, 2025
7 checks passed
@TkDodo TkDodo deleted the feat-no-void-query-fn-2 branch April 7, 2025 09:17
tylersayshi pushed a commit to tylersayshi/tanstack-query that referenced this pull request Apr 8, 2025
* feat(eslint-plugin-query): implement no-void-query-fn rule

* add docs

* ci: apply automated fixes

* build(eslint-plugin-query): switch from vite to tsup for building

* update lockfile

* revert lockfile

* fix

* fix publint

* chore: add script to patch missing export equals

* ci: apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
documentation Improvements or additions to documentation package: eslint-plugin-query
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants