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.

Sign up for GitHub

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

Jump to bottom

Very slow scanning when having symlinks folders #210

Closed
czprz opened this issue Mar 30, 2023 · 0 comments · Fixed by #211
Closed

Very slow scanning when having symlinks folders #210

czprz opened this issue Mar 30, 2023 · 0 comments · Fixed by #211
Assignees
@czprz
Labels
bug Something isn't working
Milestone
0.14.0

Comments

@czprz
Copy link
Owner

czprz commented Mar 30, 2023

Describe the bug
The more symlinks folder the slower the scan, and when using pnpm this will be so slow that it will basically never pass.

To Reproduce

  1. Have a lot of UI repos using pnpm with symlinks
  2. run 'dever scan'

Expected behavior
That scanning takes no longer whther we have symlinks or not
@czprz czprz added the bug Something isn't working label Mar 30, 2023
@czprz czprz added this to the 0.14.0 milestone Mar 30, 2023
@czprz czprz self-assigned this Mar 30, 2023
czprz added a commit that referenced this issue Mar 30, 2023
@czprz
#210 Replaced dever.json scanner with nodejs version which hopefully …
fa6e95e 
…is less sensitive to symlinks
@czprz czprz mentioned this issue Mar 30, 2023
Merged
@czprz czprz linked a pull request Mar 30, 2023 that will close this issue
#210 Replaced powershell dever.json scanner with nodejs version which hopefully is less sensitive to symlinks #211
Merged
czprz added a commit that referenced this issue Apr 6, 2023
@czprz
#210 Added scanning support for linux (testing needed)
9eac9a4
@czprz czprz closed this as completed in #211 Apr 6, 2023
czprz added a commit that referenced this issue Apr 6, 2023
@czprz
#210 Replaced dever.json scanner with nodejs version which hopefully …
0ed9db1 
…is less sensitive to symlinks
czprz added a commit that referenced this issue Apr 6, 2023
@czprz
#210 Added scanning support for linux (testing needed)
f49fe90
@czprz czprz mentioned this issue Apr 6, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@czprz czprz

Labels
bug Something isn't working
Projects
None yet
Milestone
0.14.0
1 participant
@czprz