Skip to content

Conversation

matouskozak
Copy link
Member

@matouskozak matouskozak commented Oct 13, 2025

Partial revert of 41e2b8c due to bad user experience with being unable to run the runtime-ioslike pipeline on changes affecting files outside of the trigger scope. The other option was to do refactor like this #111967 but that added an insignificant amount of complexity and was never approved thus reverting the original change and going back to the previous behavior.

Leaving the runtime-llvm pipeline trigger as is.

fyi: this need to be paired with changing the PR trigger on azdo pipeline to not run this pipeline on all PRs.

@matouskozak matouskozak self-assigned this Oct 13, 2025
@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 07:24
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reverts a previous change that restricted the runtime-ioslike pipeline to run only on PRs affecting specific Mono LLVM-related files. The change removes automatic PR triggering to improve user experience by allowing manual pipeline execution for any changes.

  • Removes automatic PR trigger configuration that was limited to specific file paths
  • Returns to manual trigger-only behavior for the runtime-ioslike pipeline

Copy link
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak, @simonrozsival
See info in area-owners.md if you want to be subscribed.

@matouskozak matouskozak added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Oct 13, 2025
@matouskozak matouskozak removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Oct 13, 2025
@matouskozak
Copy link
Member Author

/azp run runtime-ioslike

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@matouskozak
Copy link
Member Author

/azp run list

Copy link

No pipelines are associated with this pull request.

@matouskozak
Copy link
Member Author

/azp run runtime-maccatalyst

Copy link

No pipelines are associated with this pull request.

@matouskozak matouskozak merged commit 3149da8 into dotnet:main Oct 13, 2025
150 of 163 checks passed
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