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

Prepare the FabricBot config for migration to Policy Service #20540

Merged
merged 2 commits into from
Feb 26, 2024

Conversation

jeffhandley
Copy link
Member

Building on top of #20527, this updates the FabricBot configuration to address issues we've seen migrating other repositories to the Policy Service automation. There are no behavioral changes in this PR.

  1. For issue comment events, explicitly filter to match issues and not pull requests. This was previously achieved via the event_type setting, but after the port, the explicit filter will be needed.
  2. Remove references to project events.

/cc @mkArtakMSFT @wtgodbe

@jeffhandley jeffhandley requested a review from a team as a code owner February 13, 2024 05:21
@ghost ghost added the community ✨ Community Contribution label Feb 13, 2024
@ghost
Copy link

ghost commented Feb 13, 2024

Hey there @jeffhandley! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@Eilon Eilon added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Feb 21, 2024
@jfversluis jfversluis merged commit b5e8d92 into dotnet:main Feb 26, 2024
4 checks passed
rmarinho pushed a commit that referenced this pull request Feb 27, 2024
* Explicitly filter to issues for comment events

* Remove references to project events
@jeffhandley jeffhandley deleted the jeffhandley/fabricbot-prep branch March 5, 2024 20:51
@github-actions github-actions bot locked and limited conversation to collaborators Apr 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants