Skip to content

[CI] Make the API diff be triggered as soon as the config of the build is done.#21880

Merged
mandel-macaque merged 1 commit intomainfrom
dev/mandel/cascade-api-diff
Jan 1, 2025
Merged

[CI] Make the API diff be triggered as soon as the config of the build is done.#21880
mandel-macaque merged 1 commit intomainfrom
dev/mandel/cascade-api-diff

Conversation

@mandel-macaque
Copy link
Copy Markdown
Contributor

This allows the pipeline to be triggered by the build allowing us to not need to tell azp to run apidiff and simply use '/azp run xamarin-macios-pr'

…d is done.

The allows the pipeline to be triggered by the build allowing use to not
need to tell azp to run apidiff and simply use '/azp run xamarin-macios-pr'
@mandel-macaque
Copy link
Copy Markdown
Contributor Author

/azp run xamarin-macios-pr

@mandel-macaque
Copy link
Copy Markdown
Contributor Author

we should see the api diff comment without triggering the pipeline

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@vs-mobiletools-engineering-service2
Copy link
Copy Markdown
Collaborator

✅ [CI Build] Build passed (Build packages) ✅

Pipeline on Agent
Hash: 45da7bf16b30c1711b589d739b640db74399186a [PR build]

@vs-mobiletools-engineering-service2
Copy link
Copy Markdown
Collaborator

🔥 [CI Build] Build failed (Build macOS tests) 🔥

Build failed for the job 'Build macOS tests' (with job status 'Canceled')

Pipeline on Agent
Hash: 45da7bf16b30c1711b589d739b640db74399186a [PR build]

@vs-mobiletools-engineering-service2
Copy link
Copy Markdown
Collaborator

🚀 [CI Build] Test results 🚀

Test results

✅ All tests passed on VSTS: test results.

🎉 All 107 tests passed 🎉

Tests counts

✅ cecil: All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (iOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (MacCatalyst): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (macOS): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (Multiple platforms): All 1 tests passed. Html Report (VSDrops) Download
✅ dotnettests (tvOS): All 1 tests passed. Html Report (VSDrops) Download
✅ framework: All 2 tests passed. Html Report (VSDrops) Download
✅ fsharp: All 4 tests passed. Html Report (VSDrops) Download
✅ generator: All 5 tests passed. Html Report (VSDrops) Download
✅ interdependent-binding-projects: All 4 tests passed. Html Report (VSDrops) Download
✅ introspection: All 4 tests passed. Html Report (VSDrops) Download
✅ linker: All 40 tests passed. Html Report (VSDrops) Download
✅ monotouch (iOS): All 8 tests passed. Html Report (VSDrops) Download
✅ monotouch (MacCatalyst): All 10 tests passed. Html Report (VSDrops) Download
✅ monotouch (macOS): All 9 tests passed. Html Report (VSDrops) Download
✅ monotouch (tvOS): All 8 tests passed. Html Report (VSDrops) Download
✅ msbuild: All 2 tests passed. Html Report (VSDrops) Download
✅ xcframework: All 4 tests passed. Html Report (VSDrops) Download
✅ xtro: All 1 tests passed. Html Report (VSDrops) Download

Pipeline on Agent
Hash: 45da7bf16b30c1711b589d739b640db74399186a [PR build]

@mandel-macaque mandel-macaque merged commit 52975e3 into main Jan 1, 2025
@mandel-macaque mandel-macaque deleted the dev/mandel/cascade-api-diff branch January 1, 2025 16:00
rolfbjarne added a commit that referenced this pull request Apr 30, 2026
Unfortunately Azure DevOps doesn't properly report GitHub checks for pipelines
triggered by another pipeline, when that other pipeline was triggered from a
pr trigger.

So go back to triggering the API diff pipeline using a pr: trigger.

This effectively reverts #21880 ("[CI] Make the API diff be triggered as soon as the config of the build is done.")

References:

* https://stackoverflow.com/questions/78443654/reporting-stage-statuses-to-github-for-pipeline-triggered-by-another-pipeline
rolfbjarne added a commit that referenced this pull request Apr 30, 2026
Unfortunately Azure DevOps doesn't properly report GitHub checks for pipelines
triggered by another pipeline, when that other pipeline was triggered from a
pr trigger.

So go back to triggering the API diff pipeline using a pr: trigger.

This effectively reverts #21880 ("[CI] Make the API diff be triggered as soon as the config of the build is done.")

References:

* https://stackoverflow.com/questions/78443654/reporting-stage-statuses-to-github-for-pipeline-triggered-by-another-pipeline
rolfbjarne added a commit that referenced this pull request Apr 30, 2026
Unfortunately Azure DevOps doesn't properly report GitHub checks for pipelines triggered by another pipeline, when that other pipeline was triggered from a pr trigger.

So go back to triggering the API diff pipeline using a pr: trigger.

This effectively reverts #21880 ("[CI] Make the API diff be triggered as soon as the config of the build is done.")

References:

* https://stackoverflow.com/questions/78443654/reporting-stage-statuses-to-github-for-pipeline-triggered-by-another-pipeline
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.

2 participants