Skip to content

Allow 'is' for static classes for compatibility#331

Merged
gafter merged 1 commit intodotnet:masterfrom
gafter:fix324
Feb 10, 2015
Merged

Allow 'is' for static classes for compatibility#331
gafter merged 1 commit intodotnet:masterfrom
gafter:fix324

Conversation

@gafter
Copy link
Member

@gafter gafter commented Feb 9, 2015

but continue to report the diagnostic if "strict" mode is requested.
Fixes #324

@gafter gafter self-assigned this Feb 9, 2015
@gafter
Copy link
Member Author

gafter commented Feb 9, 2015

Hey, can a couple of you please look at this @jaredpar @AlekseyTs @VladimirReshetnikov @VSadov @agocke

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this comment be adjusted too?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes

@theoy theoy added the Bug label Feb 10, 2015
@gafter
Copy link
Member Author

gafter commented Feb 10, 2015

Ping for review, please @jaredpar @AlekseyTs @VladimirReshetnikov @VSadov @agocke

@gafter gafter added the 4 - In Review A fix for the issue is submitted for review. label Feb 10, 2015
@AlekseyTs
Copy link
Contributor

Is the change in Roslyn.sln intentional? Otherwise LGTM.

@jaredpar
Copy link
Member

LGTM

but continue to report the diagnostic if "strict" mode is requested.
Fixes dotnet#324
@gafter
Copy link
Member Author

gafter commented Feb 10, 2015

Reverting the change to Roslyn.sln, and squashing the commits.

gafter added a commit that referenced this pull request Feb 10, 2015
Allow 'is' for static classes for compatibility. Fixes #324.
@gafter gafter merged commit cb6761e into dotnet:master Feb 10, 2015
@gafter gafter deleted the fix324 branch February 10, 2015 21:13
@gafter gafter removed the 4 - In Review A fix for the issue is submitted for review. label Feb 10, 2015
dibarbet pushed a commit to dibarbet/roslyn that referenced this pull request Nov 21, 2025
Take care to use proper slashes per OS for testing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CodePlex] Error CS7023 'is' for static classes [breaking compiler change]

4 participants