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

Ability to ignore certain folders from CodeLens or References #3884

Closed
2 tasks done
PrzemyslawKlys opened this issue Mar 21, 2022 · 5 comments
Closed
2 tasks done

Ability to ignore certain folders from CodeLens or References #3884

PrzemyslawKlys opened this issue Mar 21, 2022 · 5 comments
Assignees
Labels
Area-Configuration Issue-Enhancement A feature request (enhancement). Resolution-Fixed Will close automatically.

Comments

@PrzemyslawKlys
Copy link
Contributor

Prerequisites

  • I have written a descriptive issue title.
  • I have searched all issues to ensure it has not already been reported.

Summary

I would like to be able to ignore certain folders from being used by CodeLens, References, and so on.

During the building of my module, I often have additional folders called Releases or Artifacts where I push built modules. Sometimes it's just one version, sometimes 50 last version and this affects some things:

  1. It slows down PowerShell Features
  2. It's picked up by references
  3. Jumping to function often jumps to built code rather than to the "expected" function.

When excluded those PS1 files would be skipped and would improve general performance.

Proposed Design

Some option in JSON to exclude $PSScriptRoot\Artifacts or $PSScriptRoot\Releases

@PrzemyslawKlys PrzemyslawKlys added the Issue-Enhancement A feature request (enhancement). label Mar 21, 2022
@ghost ghost added the Needs: Triage Maintainer attention needed! label Mar 21, 2022
@andyleejordan andyleejordan added Area-Configuration and removed Needs: Triage Maintainer attention needed! labels Mar 22, 2022
@andyleejordan
Copy link
Member

Related, similar feature request: #3048

@andyleejordan
Copy link
Member

Also related to #3902.

@SydneyhSmith
Copy link
Collaborator

This will be incorporated into references refactor work

@SeeminglyScience
Copy link
Collaborator

Support for this was already added some time ago (though tbh I only noticed while redoing some of the references code), it uses the existing setting search.exclude. For example, in my profile directory I have this settings.json:

{
    "search.exclude": {
        "**/node_modules": true,
        "**/bower_components": true,
        "**/*.code-search": true,
        "/Modules": true,
    }
}

Which excludes all of my installed modules from workspace analysis.


That said, I did also make it a lot faster, and the cost is all upfront instead of every code lens refresh. So may not be needed, but if it is, it's available!

@ghost
Copy link

ghost commented Dec 19, 2022

Thank you for your comment, but please note that this issue has been closed for over a week. For better visibility, consider opening a new issue with a link to this instead.

@SeeminglyScience SeeminglyScience added the Resolution-Fixed Will close automatically. label Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Configuration Issue-Enhancement A feature request (enhancement). Resolution-Fixed Will close automatically.
Projects
None yet
Development

No branches or pull requests

4 participants