Skip to content
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

Enable public Azure DevOps builds #366

Merged
merged 5 commits into from
Nov 22, 2022
Merged

Enable public Azure DevOps builds #366

merged 5 commits into from
Nov 22, 2022

Conversation

dougbu
Copy link
Member

@dougbu dougbu commented Nov 21, 2022

  • add azure-pipelines.yml
    • build about the same matrix as we have on TeamCity in Main.Integration builds
  • change build.cmd to enable overrides of its command line parameters

nits:

  • touch up .codeql.yml to avoid test code and include portable projects

- add azure-pipelines.yml
  - clean workspace when checking out the repo
  - build about the same matrix as we have on TeamCity in Main.Integration builds
    - run tests only in Release job by default
  - do component governance work only in Release job
  - do not inject CodeQL3000 tasks (when building main)
  - publish test results
- change build.cmd to enable overrides of its command line parameters

nits:
- touch up .codeql.yml to avoid test code and include portable projects
- reduce job timeout; jobs usually take less than 10 minutes
- make test run title more readable / useful
@dougbu dougbu merged commit 0b0f6bf into main Nov 22, 2022
@dougbu dougbu deleted the dougbu/pr.pipeline branch November 22, 2022 22:33
Copy link

@TanayParikh TanayParikh left a comment

Choose a reason for hiding this comment

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

Don't have too much context in this, but looks good overall

@dougbu dougbu added this to the 3.3.0 (5.3.0) milestone Feb 26, 2023
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