Skip to content

Conversation

@dougbu
Copy link
Contributor

@dougbu dougbu commented Aug 31, 2020

  • used $(IsUnitTestProject) in src/Razor/test; slightly more correct

@dougbu dougbu requested review from a team, NTaylorMullen, ajaybhargavb and pranavkm August 31, 2020 20:21
@dougbu dougbu requested review from a team and SteveSandersonMS as code owners August 31, 2020 20:21
@dougbu
Copy link
Contributor Author

dougbu commented Aug 31, 2020

@Pilchie this changes what we ship. So I'm asking for your approval. Please…

Copy link
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

Looks good pending the 1 question

@Pilchie
Copy link
Member

Pilchie commented Aug 31, 2020

Seems good to me. Approved for RC1 pending CI

dougbu added 2 commits August 31, 2020 15:13
- used `$(IsUnitTestProject)` in src/Razor/test; slightly more correct
- don't seem to have any actual tests in this folder (unless I missed something)
@dougbu dougbu force-pushed the dougbu/non-shipping.non-packable branch from 2746b63 to 6e70b9e Compare August 31, 2020 22:35
@dougbu
Copy link
Contributor Author

dougbu commented Sep 1, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@dougbu dougbu merged commit 7579b30 into release/5.0 Sep 1, 2020
@dougbu dougbu deleted the dougbu/non-shipping.non-packable branch September 1, 2020 05:46
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.

6 participants