Skip to content

Re-unskip code model tests#6021

Merged
DustinCampbell merged 2 commits intodotnet:masterfrom
DustinCampbell:unskip-code-model-tests
Oct 16, 2015
Merged

Re-unskip code model tests#6021
DustinCampbell merged 2 commits intodotnet:masterfrom
DustinCampbell:unskip-code-model-tests

Conversation

@DustinCampbell
Copy link
Member

It appears that a handful of code model tests that were previously skipped and then unskipped, were inadvertently re-skipped by commit: 978df2e.

This change puts things back the way they were before.

Tagging @dotnet/roslyn-ide, @jaredpar

@jaredpar
Copy link
Member

👍

1 similar comment
@Pilchie
Copy link
Member

Pilchie commented Oct 15, 2015

👍

@DustinCampbell
Copy link
Member Author

test linux please

@DustinCampbell
Copy link
Member Author

test mac please

@DustinCampbell
Copy link
Member Author

test eta please

DustinCampbell added a commit that referenced this pull request Oct 16, 2015
@DustinCampbell DustinCampbell merged commit be589cc into dotnet:master Oct 16, 2015
@DustinCampbell DustinCampbell deleted the unskip-code-model-tests branch December 14, 2015 21:19
github-actions bot pushed a commit that referenced this pull request Apr 1, 2025
#6272)

* Added support for StringSyntaxAttribute.CompositeFormat in CA2241 (#6021)

* Use a ConcurrentDictionary for methods investigated.

This way we only need to check every method definition once.
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.

4 participants