Skip to content

Get rid of unused Version.csproj.include#19679

Merged
jtattermusch merged 1 commit intogrpc:masterfrom
jtattermusch:cleanup_version_csproj
Jul 19, 2019
Merged

Get rid of unused Version.csproj.include#19679
jtattermusch merged 1 commit intogrpc:masterfrom
jtattermusch:cleanup_version_csproj

Conversation

@jtattermusch
Copy link
Copy Markdown
Contributor

@jtattermusch jtattermusch commented Jul 18, 2019

In https://github.com/grpc/grpc/pull/18486/files, Version.csproj.include was replaced with dependencies.props, but it wasn't really removed and we ended up with a dead copy of Version.csproj.include which isn't used from anywhere (but it was actually updated by some PRs by accident, making it looks that Google.Protobuf dependency version was updated when it really wasn't).

This PR removes the dead copy and makes dependencies.props up to date.

@jtattermusch jtattermusch added lang/C# release notes: no Indicates if PR should not be in release notes labels Jul 18, 2019
@jtattermusch
Copy link
Copy Markdown
Contributor Author

CC @anandolee

@jtattermusch jtattermusch force-pushed the cleanup_version_csproj branch from 566e1e2 to c767398 Compare July 19, 2019 15:08
@jtattermusch jtattermusch merged commit 00d38d7 into grpc:master Jul 19, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lang/C# release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants