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

Code Inspector not excluding Ignored Projects #6030

Open
skorpio07 opened this issue Oct 13, 2022 · 3 comments
Open

Code Inspector not excluding Ignored Projects #6030

skorpio07 opened this issue Oct 13, 2022 · 3 comments
Labels
bug Identifies work items for known bugs
Milestone

Comments

@skorpio07
Copy link

Rubberduck version information
The info below can be copy-paste-completed from the first lines of Rubberduck's log or the About box:

Version 2.5.2.5906
OS: Microsoft Windows NT 10.0.19044.0, x64
Host Product: Microsoft Office x86
Host Version: 16.0.14931.20724
Host Executable: EXCEL.EXE

Description
When I run Code Inspections, it includes the addins from the Ignored Projects.

To Reproduce
Steps to reproduce the behavior:

  1. Add an Ignored Project
  2. Run or Refresh CI
  3. View errors and note that the Ignored Projects addin is included in the CI report.

Expected behavior
For CI to exclude the addins listed in the Ignored Projects

Screenshots
Not applicable

Logfile
Included
RubberduckLog.txt

Additional context
Nothing

Add any other context about the problem here.

@skorpio07 skorpio07 added the bug Identifies work items for known bugs label Oct 13, 2022
@retailcoder
Copy link
Member

Likely a cache invalidation issue, assuming the project wasn't locked during the initial parse. Adding an ignored project should clear it from the cache.
If the project is already ignored at startup, it doesn't produce inspection results, does it?

@skorpio07
Copy link
Author

skorpio07 commented Oct 13, 2022 via email

@retailcoder
Copy link
Member

Yup, at startup it shows results for ignored projects.

Interesting, that's definitely a bug... Ignored projects shouldn't be parsed, that was the whole idea.

Not sure if you need any help but I am pretty good at VBA. If there is anyway to lend you a hand doing that, then please consider me.

There isn't much VBA in Rubberduck (it's written in C#), but if you're feeling adventurous you can clone the repository, explore the source code (use VS 2019, the solution won't build in VS 2022), ask any questions you have and we'll be happy to guide you towards your fist pull request! We don't pick contributors, contributors pick Rubberduck ☺️

@retailcoder retailcoder added this to the 2.x Cycle milestone Jan 31, 2025
@retailcoder retailcoder moved this to Backlog | Inspections in Board-v2.99 Feb 2, 2025
@retailcoder retailcoder moved this from Backlog | Inspections to Ready in Board-v2.99 Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Identifies work items for known bugs
Projects
Status: Ready
Development

No branches or pull requests

2 participants