-
Notifications
You must be signed in to change notification settings - Fork 934
Upgrade AsyncGenerator to 0.14.0 #2126
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
Conversation
#if NETFX | ||
[Test, Ignore("Not fixed yet.")] | ||
[Description("SqlClient: The batcher should run all different INSERT queries in only one roundtrip.")] | ||
public async Task SqlClientOneRoundTripForUpdateAndInsertAsync() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't get it why this was removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Until now the generator ignored only tests that had the Ignore
attribute on the class, now I've applied the same rule also for methods.
I've changed to WIP until the issue in nhibernate/NHibernate-Caches/pull/65 is resolved. For some reason the issue does not happen for this project even when regenerating multiple times. |
Removed WIP as the mentioned issue is not related to the generator. |
A conflict has appeared in the meantime, if you have not seen it. |
UPD: My bad, I had a dirty branch. |
The new version contains an updated NUnit plugin that simplifies the integration with NUnit.