fix(sherif): remove unnecessary option and ignore directory

[manudeli]

@TkDodo I saw your feature request in QuiiBz/sherif#41 and feature is added in v7.0.0 of sherif
Thanks to request feature to sherif. I am using this tool and this I needed this feature too.

To resolve your comment in #6686 I updated script for sherif. from now we don't have to use unnecessary option(-i react-scripts -i typescript) So cool👍

[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)	Visit Preview		Jan 14, 2024 6:11pm

[nx-cloud]

☁️ Nx Cloud Report

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

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 1 target

• nx affected --targets=test:eslint,test:lib,test:types,test:build,build --exclude=examples/**

---

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 0d7b08c:

Sandbox	Source
@tanstack/query-example-angular-basic	Configuration
@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

All modified and coverable lines are covered by tests ✅

Comparison is base (3a127bf) 41.71% compared to head (0d7b08c) 41.71%.
Report is 2 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6701   +/-   ##
=======================================
  Coverage   41.71%   41.71%           
=======================================
  Files         179      179           
  Lines        7007     7007           
  Branches     1416     1416           
=======================================
  Hits         2923     2923           
  Misses       3713     3713           
  Partials      371      371           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[TkDodo]

not sure how but this is failing on main:

https://cloud.nx.app/runs/oblIhu1Dfj

> query@ test:sherif /home/runner/work/query/query
> sherif --ignore-package ./integrations/*

error: unexpected argument './integrations/react-cra5' found

Usage: sherif [OPTIONS] [PATH]

Can you please investigate and fix it, or I need to revert. Thanks

[manudeli]

not sure how but this is failing on main:

https://cloud.nx.app/runs/oblIhu1Dfj

> query@ test:sherif /home/runner/work/query/query
> sherif --ignore-package ./integrations/*

error: unexpected argument './integrations/react-cra5' found

Usage: sherif [OPTIONS] [PATH]

Can you please investigate and fix it, or I need to revert. Thanks

@TkDodo Revert first please not to affect other branches working, I will investigate it soon

[manudeli]

@TkDodo, I started to investigate this issue in QuiiBz/sherif#51

[QuiiBz]

Hi, seems like your shell automatically interprets and expand *, so the resulting command is indeed incorrect. In that case, you have to wrap the argument in quotes: sherif --ignore-package "./integrations/*"

Note that this isn't related to Sherif directly, but I'll update the documentation to include these quotes.