Skip to content

Change CA2249 to warning#2040

Merged
slozier merged 2 commits into
IronLanguages:mainfrom
slozier:ca2249
May 13, 2026
Merged

Change CA2249 to warning#2040
slozier merged 2 commits into
IronLanguages:mainfrom
slozier:ca2249

Conversation

@slozier
Copy link
Copy Markdown
Contributor

@slozier slozier commented May 12, 2026

This remove CA2249 from .editorconfig (thus changing it to warning) and ensures the polyfill is available.

@BCSharp
Copy link
Copy Markdown
Member

BCSharp commented May 12, 2026

You may also want to update the Meizantou.Polyfill version at the same time, like the DLR. Polyfill Contains calls IndexOf, which in v.1.0.125+ has been updated with the IndexOf from the ex-DLR polyfill.

@slozier slozier merged commit 269d3ae into IronLanguages:main May 13, 2026
13 of 26 checks passed
@slozier slozier deleted the ca2249 branch May 13, 2026 00:38
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.

2 participants