Skip to content

fix:Analyzer false positive on Indexed CollectionFormat - #2274

Merged
glennawatson merged 4 commits into
reactiveui:mainfrom
jgarciadelanoceda:issue_2273_AnalyzerIndexed
Jul 24, 2026
Merged

fix:Analyzer false positive on Indexed CollectionFormat#2274
glennawatson merged 4 commits into
reactiveui:mainfrom
jgarciadelanoceda:issue_2273_AnalyzerIndexed

Conversation

@jgarciadelanoceda

@jgarciadelanoceda jgarciadelanoceda commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

What kind of change does this PR introduce?

Fixes #2273

What is the new behavior?

The new behaviour doesn´t emit the RF006

What is the current behavior?

It emitted it provoking a false positive on the analyser

What might this PR break?

N/A

Checklist

  • I have read the Contribute guide
  • Tests have been added or updated (for bug fixes / features)
  • Docs have been added or updated (for bug fixes / features)
  • Changes target the main branch
  • PR title follows Conventional Commits

Additional information

I have also seen that the CodeQL task when it was merged in master it throw that some paths were not covered and I covered them. Also remove some analyser warnings that was done on #2268, but my original PR was merged after this one

Comment thread src/InterfaceStubGenerator.Shared/Parser.Request.Query.cs
@glennawatson
glennawatson merged commit 8e4237b into reactiveui:main Jul 24, 2026
13 checks passed
@jgarciadelanoceda
jgarciadelanoceda deleted the issue_2273_AnalyzerIndexed branch July 24, 2026 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: CollectionFormat.Indexed is putting a false analyser positive

2 participants