Skip to content

Skip post-build stages for validation build#44298

Merged
2 commits merged intodotnet:masterfrom
genlu:fixYaml
May 16, 2020
Merged

Skip post-build stages for validation build#44298
2 commits merged intodotnet:masterfrom
genlu:fixYaml

Conversation

@genlu
Copy link
Member

@genlu genlu commented May 15, 2020

No description provided.

@genlu genlu marked this pull request as ready for review May 15, 2020 23:21
@genlu genlu requested a review from a team as a code owner May 15, 2020 23:21
# since we don't want to publish validation build.
validateDependsOn:
- SetValidateDependency
dependsOn:
Copy link
Member Author

Choose a reason for hiding this comment

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

We set both because arcade 1.0 uses dependsOn and arcade 5.0 uses validateDependsOn, and I'm not certain why we are switching back and force.

Copy link
Member Author

Choose a reason for hiding this comment

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

Also I found it's odd that AzDo doesn't complain about assigning value to an non-existing template parameter. Not sure if there's a reason behind that or just simply an oversight. @tmat Whom could I talk to about this?

@genlu genlu requested review from JoeRobich and tmat May 15, 2020 23:24
Copy link
Member

@JoeRobich JoeRobich left a comment

Choose a reason for hiding this comment

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

Be sure to link to your PR validation run. =)

Copy link
Member Author

@genlu genlu left a comment

Choose a reason for hiding this comment

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

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

@ghost ghost merged commit c8eca69 into dotnet:master May 16, 2020
@genlu genlu deleted the fixYaml branch May 18, 2020 18:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants