Skip to content

Conversation

@tpaxatb
Copy link
Contributor

@tpaxatb tpaxatb commented Mar 10, 2020

To allow for the generated files to be available during the first pass,
they must be generated before the pass1. In addition, the files that are
generated must be added to the _GeneratedCodeFiles items.

Fixes #2167.

To allow for the generated files to be available during the first pass,
they must be generated before the pass1.  In addition, the files that are
generated must be added to the _GeneratedCodeFiles items.
Copy link
Member

@asbjornu asbjornu left a comment

Choose a reason for hiding this comment

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

LGTM, but how is our test coverage around this? @arturcic, do you know?

@arturcic
Copy link
Member

Currently the tests we have don't cover the order of the tasks, and to be honest I don't know if we can automate that.

@asbjornu
Copy link
Member

I'm more concerned with the task still working at all than the order being as expected. As long as we have tests that dogfood GitVersionTask and check the generated version variables, I think we're good.

@arturcic
Copy link
Member

We do have tests that run the msbuild artifact and checks the generated version

@asbjornu asbjornu merged commit 1707bbc into GitTools:master Mar 12, 2020
@asbjornu
Copy link
Member

@arturcic, great! Thanks for your contributions, @tpaxatb! 🙏

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.

UpdateAssemblyInfo and GenerateGitVersionInformation need to be done earlier

3 participants