Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[corlib] Bring back String.Concat with __arglist #10452

Merged
merged 4 commits into from Sep 4, 2018

Conversation

EgorBo
Copy link
Member

@EgorBo EgorBo commented Sep 3, 2018

Fixes #9996
Tested repro project from the link on a local PostgresSQL server and EntityFramework packages.

See https://github.com/aspnet/EntityFramework6/blob/master/src/EntityFramework/Core/Objects/ELinq/MethodCallTranslator.cs#L1482-L1497

@jaykrell
Copy link
Contributor

jaykrell commented Sep 3, 2018

Test case?

@EgorBo
Copy link
Member Author

EgorBo commented Sep 3, 2018

@jaykrell Added!

@marek-safar
Copy link
Member

@monojenkins backport 2018-08

@marek-safar
Copy link
Member

@monojenkins backport 2018-06

@marek-safar
Copy link
Member

@monojenkins backport 2018-04

@marek-safar
Copy link
Member

@monojenkins commit apidiff

monojenkins added a commit to mono/api-snapshot that referenced this pull request Sep 4, 2018
@EgorBo EgorBo requested a review from luhenry as a code owner September 4, 2018 07:39
@marek-safar marek-safar merged commit 0a4aa46 into mono:master Sep 4, 2018
akoeplinger pushed a commit that referenced this pull request Sep 4, 2018
akoeplinger pushed a commit that referenced this pull request Sep 4, 2018
akoeplinger pushed a commit that referenced this pull request Sep 4, 2018
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.

None yet

5 participants