Skip to content

Upgrade AsyncGenerator to 0.13.1 #1936

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

Merged
merged 3 commits into from
Jan 24, 2019

Conversation

fredericDelaporte
Copy link
Member

@fredericDelaporte fredericDelaporte commented Dec 11, 2018

NHibernate currently used a 0.8 version having received many backports of fixes from newer versions.

I think we should upgrade it to latest version instead of going on relying on backports in case of new needs / new unsupported patterns used in NHibernate.

This update also refines the cases where generated async code is guarded by try catch clauses. (Some are removed, other are added.)

By the way, I let the .DotSettings file get updated by Rider here: it does no more remove the property we were not willing to let it remove.

@fredericDelaporte fredericDelaporte changed the title Upgrade AsyncGenerator to 0.13.0 WIP - Upgrade AsyncGenerator to 0.13.0 Dec 11, 2018
@fredericDelaporte fredericDelaporte changed the title WIP - Upgrade AsyncGenerator to 0.13.0 Upgrade AsyncGenerator to 0.13.0 Dec 11, 2018
@fredericDelaporte fredericDelaporte changed the title Upgrade AsyncGenerator to 0.13.0 Upgrade AsyncGenerator to 0.13.1 Dec 11, 2018
hazzik
hazzik previously approved these changes Jan 23, 2019
@fredericDelaporte fredericDelaporte merged commit f7f783e into nhibernate:master Jan 24, 2019
@fredericDelaporte fredericDelaporte deleted the AsyncUpgrade branch January 24, 2019 22:08
@fredericDelaporte
Copy link
Member Author

@maca88, @bahusoid, having merged this PR may cause some async generation changes in pending PR. So better updates them and check an async generation.

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.

3 participants