microsoft / microsoft/ApplicationInspector
Support TagsOnly mode with DependsOnTags
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 4.4k
- Forks
- 366
- Avg merge
- 6d 7h
- Merged PRs (30d)
- 3
Description
Is your feature request related to a problem? Please describe.
TagsOnly mode currently is not compatible with the DependsOnTags feature. This is because TagsOnly mode does not collect the full match data necessary for the current DependsOnTags implementation.
Describe the solution you'd like
TagsOnly mode could collect tags as a mapping between Tags and a List of rules that apply that tag. When entering depends on checking, if in tags only mode we can perform a similar rule removing process with recalculating the tags.
Describe alternatives you've considered
Full match data could still be gathered and just discarded later instead, but this will negatively impact performance.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing the existing TagsOnly and DependsOnTags implementations and how TagsOnly currently collects match data. Determine how tags can retain the rules that apply to them, then verify that dependency filtering and tag recalculation work without collecting full match data. Done means TagsOnly supports DependsOnTags without the proposed performance cost.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- security
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100