search

ni / semi-test-library-dotnet

Semiconductor Test Library
https://ni.github.io/semi-test-library-dotnet/
MIT License
3 stars 2 forks source link

Improvements in new workflow after first round testing #159

Closed Tanish-NI closed 1 week ago

Tanish-NI commented 1 week ago

What does this Pull Request accomplish?

  1. Replacing github.event.workflow_run.pull_requests[0].number with github.event.workflow_run.run_number to use run_number as identifier in pr-intake instead of pr_number as pr_number is not retrieving correctly..
  2. Arguments we pass to the AzDo pipeline from GitHub workflows need to be updated to support building code from forked repos. Currently, we are just passing branch-name which is not sufficient in case of forked repos. We should pass 3 things:- repo owner, repo name, and repo branch. Using these 3 things, we will be able to clone the correct repo we want to build and test using AzDo pipeline.

Why should this Pull Request be merged?

  1. Updated pr-intake, pr-build, and pr-test workflow.
  2. Used run_number as an identifier of pr-build and pr-test runs to fetched in pr-intake.
  3. Passed 3 parameters to AzDo pipelines: GITHUB_MixedSignalTestLibrary_REPO_OWNER, GITHUB_MixedSignalTestLibrary_REPO_NAME, GITHUB_MixedSignalTestLibrary_BRANCH_NAME.
  4. Temporary using refs/heads/users/tanish/AzDo-change-for-github-workflows-improvements AzDo branch as AzDo pipeline source to test AzDo changes.

What testing has been done?

NA