Skip to content

[5.0-preview3] Remove Code_check job from publish-build-assets.yml dependsOn#20559

Merged
dougbu merged 1 commit into
release/5.0-preview3from
dougbu/use.codecheck.less/5.0p3
Apr 5, 2020
Merged

[5.0-preview3] Remove Code_check job from publish-build-assets.yml dependsOn#20559
dougbu merged 1 commit into
release/5.0-preview3from
dougbu/use.codecheck.less/5.0p3

Conversation

@dougbu
Copy link
Copy Markdown
Contributor

@dougbu dougbu commented Apr 5, 2020

  • re-enable the "Build SiteExtension" build step!
  • nit: remove redundant conditions for easier reading

- re-enable the "Build SiteExtension" build step!
- nit: remove redundant conditions for easier reading
@dougbu dougbu added area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode labels Apr 5, 2020
@dougbu dougbu requested review from a team and BrennanConroy April 5, 2020 20:58
Comment thread .azure/pipelines/ci.yml
$(_InternalRuntimeDownloadArgs)
# Disabled until 3.1.3 is released
condition: false
condition: ne(variables['Build.Reason'], 'PullRequest')
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

FYI this step doesn't have a condition on it in 'release/3.1' branch, meaning it will run if previous steps succeed.

@dougbu dougbu merged commit d6821fa into release/5.0-preview3 Apr 5, 2020
@dougbu dougbu deleted the dougbu/use.codecheck.less/5.0p3 branch April 5, 2020 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants