Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

v7.0.1 BUG #726

Open
babakfp opened this issue Sep 24, 2023 · 6 comments
Open

v7.0.1 BUG #726

babakfp opened this issue Sep 24, 2023 · 6 comments

Comments

@babakfp
Copy link

babakfp commented Sep 24, 2023

rror: source `[]` not found before ignore pattern is found
    at file:///C:/Users/babak/OneDrive/Documents/GitHub/babakfp.ir/node_modules/.pnpm/ignore-sync@7.0.1/node_modules/ignore-sync/src/decodeIgnoreSyncFile.js:24:13
    at Array.reduce (<anonymous>)
    at decodeIgnoreSyncFile (file:///C:/Users/babak/OneDrive/Documents/GitHub/babakfp.ir/node_modules/.pnpm/ignore-sync@7.0.1/node_modules/ignore-sync/src/decodeIgnoreSyncFile.js:11:53)       
    at async processIgnoreSyncFile (file:///C:/Users/babak/OneDrive/Documents/GitHub/babakfp.ir/node_modules/.pnpm/ignore-sync@7.0.1/node_modules/ignore-sync/src/processIgnoreSyncFile.js:10:22)
    at async Promise.all (index 0)
    at async processDirectory (file:///C:/Users/babak/OneDrive/Documents/GitHub/babakfp.ir/node_modules/.pnpm/ignore-sync@7.0.1/node_modules/ignore-sync/src/processDirectory.js:43:5)
    at async startApp (file:///C:/Users/babak/OneDrive/Documents/GitHub/babakfp.ir/node_modules/.pnpm/ignore-sync@7.0.1/node_modules/ignore-sync/src/app.js:25:5)

This issue was resolved after switching to v7.0.0

@foray1010
Copy link
Owner

Can you provide a reproducible example?

@babakfp
Copy link
Author

babakfp commented Sep 24, 2023

git clone https://github.com/babakfp/babakfp.ir.git
cd babakfp.ir
git checkout ca3b1375f64ff53c65662db92025f6805fc886da

pnpm i
pnpm ignore-sync

@foray1010
Copy link
Owner

That is a private repo, can u post the config file here?

@babakfp
Copy link
Author

babakfp commented Sep 24, 2023

Ops!

Here's a proper reproduction: https://github.com/babakfp/foray1010.ignore-sync.issues.726

@foray1010
Copy link
Owner

It works fine on my side.

@babakfp
Copy link
Author

babakfp commented Sep 24, 2023

Rec.0001.mp4
NodeJS: v20.6.1
PNPM: 8.6.12
NPM: 9.8.1
Windows 11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants