Skip to content

Run workflow on feature branches#689

Merged
bmagyar merged 1 commit intoros-controls:jtc-featuresfrom
christophfroehlich:workflow_feature_branches
Jun 28, 2023
Merged

Run workflow on feature branches#689
bmagyar merged 1 commit intoros-controls:jtc-featuresfrom
christophfroehlich:workflow_feature_branches

Conversation

@christophfroehlich
Copy link
Copy Markdown
Member

@christophfroehlich christophfroehlich commented Jun 27, 2023

Rolling build workflows now should run on workflow_dispatch and pull_request events on branches matching *feature* or *feature/**.

These jobs fail now, because of #688

@destogl
Copy link
Copy Markdown
Member

destogl commented Jun 27, 2023

Is this something we really need? Usually, maintainers can restart CI quite without problems, just opening the workflow.

@christophfroehlich
Copy link
Copy Markdown
Member Author

Now it would be handy with the jtc-features branch, because the PRs targeting into it aren't checked (can you manually run them on an arbitrary PR?). For this maybe the change of workflow_dispatch is not really necessary, but pull_request would be.
But I don't now if this feature branch is an extraordinary exception, or if that might be a best-practice example for the future.

Copy link
Copy Markdown
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

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

I like the idea!!

@bmagyar bmagyar merged commit 48c6593 into ros-controls:jtc-features Jun 28, 2023
@christophfroehlich christophfroehlich deleted the workflow_feature_branches branch June 28, 2023 18:08
christophfroehlich added a commit to christophfroehlich/ros2_controllers that referenced this pull request Jul 17, 2023
@destogl
Copy link
Copy Markdown
Member

destogl commented Jul 17, 2023

Should we revert this, or do we want to merge those changes into master too?

@christophfroehlich
Copy link
Copy Markdown
Member Author

I vote for bringing this to master for other new feature branches in the future.

bmagyar pushed a commit that referenced this pull request Jul 17, 2023
mergify bot pushed a commit that referenced this pull request Jul 17, 2023
(cherry picked from commit f70c986)
mergify bot pushed a commit that referenced this pull request Jul 17, 2023
(cherry picked from commit f70c986)
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.

3 participants