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

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

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

No branches or pull requests

2 participants