-
Notifications
You must be signed in to change notification settings - Fork 132
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
change: Update project target frameworks and dependencies #251
Conversation
@hiranya911 @lahirumaramba Can someone review please? |
@DenSmoke Thanks for the PR. Really appreciate your effort. This contains several breaking changes. Specifically, following changes are breaking:
We can look into accepting those changes in the next quarter, and releasing v2 of the SDK. Can I ask you to extract the above 2 changes from here into a separate PR? The remaining changes in this PR (e.g. upgrading test dependencies) can be merged immediately. |
@hiranya911 Upgrading test dependencies is not critical. Main purpose is to solve conflicts like in issue #207 when using GAX 3.0 dependent Google Cloud .NET libraries and Firebase Admin SDK in same .NET project |
Yeah, I understand. We can hold on to this PR for now, and come up with a plan to merge it in the coming months. I just need to sync up with other teams and see if we have any upcoming releases that might potentially conflict with this. |
We have slated this for a release in the next quarter (q4 2020). As a first step we will try to add |
@DenSmoke can I ask you to update this PR and sync up with the latest state of the master branch? We can merge this soon after that. |
@hiranya911 done |
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.
Thank you @DenSmoke. This LGTM. I'll merge it in a couple of days, and start working on a release.
API CHANGE: Dropped support for
netstandard1.5
andnet45
target frameworks. Developers are now required to usenetstandard2.0
ornet461
.API CHANGE: Upgraded the dependency
Google.Api.Gax
to the latest major version. Developers that use thePagedAsyncEnumerable
interface from this package (usually as the return value of list iteration API calls), may have to update their implementations accordingly.Resolves #264 #265 #207