Skip to content

Add header dictionary analyzer and fixer#44123

Merged
JamesNK merged 1 commit into
mainfrom
jamesnk/headerdictionary-analyzer
Sep 26, 2022
Merged

Add header dictionary analyzer and fixer#44123
JamesNK merged 1 commit into
mainfrom
jamesnk/headerdictionary-analyzer

Conversation

@JamesNK
Copy link
Copy Markdown
Member

@JamesNK JamesNK commented Sep 22, 2022

Fixes #44116

image

@JamesNK JamesNK requested review from a team, Pilchie, dougbu and wtgodbe as code owners September 22, 2022 13:13
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Sep 22, 2022
Comment thread eng/Versions.props Outdated
@JamesNK
Copy link
Copy Markdown
Member Author

JamesNK commented Sep 23, 2022

Brought in changes from #44061 so I can get all tests to pass when I update the verifiers.

It's merged in release/7.0, and will get merged to main soon. Small merge conflict to resolve depending on who gets merged first 💪

@JamesNK JamesNK force-pushed the jamesnk/headerdictionary-analyzer branch from 9ae0e7c to 8b59ba0 Compare September 26, 2022 08:40
@JamesNK JamesNK force-pushed the jamesnk/headerdictionary-analyzer branch from ad5aaf4 to 76cd4d5 Compare September 26, 2022 22:03
@JamesNK JamesNK enabled auto-merge (squash) September 26, 2022 22:04
@JamesNK JamesNK merged commit daab261 into main Sep 26, 2022
@JamesNK JamesNK deleted the jamesnk/headerdictionary-analyzer branch September 26, 2022 23:41
@ghost ghost added this to the 8.0-preview1 milestone Sep 26, 2022
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an Analyzer to recommend using DIM properties with IHeaderDictionary instead of indexer

3 participants