Skip to content

Add a new job to handle push event matching #1279

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

Merged
merged 18 commits into from
May 21, 2025

Conversation

Mpdreamz
Copy link
Member

Introduce a "match" job to process push event-specific logic and outputs, ensuring proper handling of event types. This includes conditional steps for different event contexts and prepares outputs for downstream "build" job dependencies.

Introduce a "match" job to process push event-specific logic and outputs, ensuring proper handling of event types. This includes conditional steps for different event contexts and prepares outputs for downstream "build" job dependencies.
@Mpdreamz Mpdreamz requested a review from a team as a code owner May 20, 2025 13:00
@Mpdreamz Mpdreamz added the chore label May 20, 2025
@Mpdreamz Mpdreamz self-assigned this May 20, 2025
@Mpdreamz Mpdreamz added the chore label May 20, 2025
Copy link
Member

@reakaleek reakaleek left a comment

Choose a reason for hiding this comment

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

We could change on.pull_request_target to on.pull_request to test if it's working on this PR.

Without affecting other repos.

@Mpdreamz
Copy link
Member Author

@reakaleek I think this is now ready for a spin or our main and temporary 9.0 branches.

I can then follow up with making build depend on match's output.

@Mpdreamz Mpdreamz merged commit 755ba56 into main May 21, 2025
14 checks passed
@Mpdreamz Mpdreamz deleted the feature/add-match-to-worklfow branch May 21, 2025 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants