Skip to content

Fix discrepancy between Version.Details.xml and Versions.props #7314

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 2 commits into from
Feb 6, 2019

Conversation

natemcmaster
Copy link
Contributor

Somehow the versions in these two files is no longer aligned. The bot reads Version.Details.xml and the build uses Versions.props. If they're not aligned, this can cause the bot to not run as expected.

cc @pakrym

@natemcmaster natemcmaster requested a review from dougbu February 6, 2019 19:50
@natemcmaster natemcmaster merged commit 68f7e3a into dotnet:master Feb 6, 2019
@natemcmaster natemcmaster deleted the deps branch February 6, 2019 20:08

1. Create the .csproj
2. Run `eng/scripts/GenerateProjectList.ps1`
3. Add it to Extensions.sln
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pick a different solution and mention it's an example. We're in aspnet/AspNetCore and that repo doesn't have an Extensions.sln file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merged before I saw this feedback. I'll update in my next PR

<Uri>https://github.com/aspnet/Extensions</Uri>
<Sha>9750bf67ab22d8d28255c6222be282f4c8e49383</Sha>
<Sha>000</Sha>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest making this (000) and the document (000000) consistent

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will get blown away by the next bot update (#7301). I merged this fix as is to unblock the automation that was incorrectly generating PRs to this repo.

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.

3 participants