Skip to content

Move to GitFlow strategy [ci patch inc]#6

Closed
HyperWinX wants to merge 1 commit intomasterfrom
move-to-gitflow
Closed

Move to GitFlow strategy [ci patch inc]#6
HyperWinX wants to merge 1 commit intomasterfrom
move-to-gitflow

Conversation

@HyperWinX
Copy link
Member

Update workflows to comply with GitFlow strategy.

@mentatbot
Copy link

mentatbot bot commented Apr 4, 2025

I'm getting to work. See my steps and track my cost here.

  • Wake on any new activity.
    I always see new activity while actively working. When paused, I'll only respond when tagged unless this box is checked.

@mentatbot
Copy link

mentatbot bot commented Apr 4, 2025

I've reviewed the changes to move to GitFlow strategy. The workflow updates look clean and well-structured, with several improvements:

  • Simplified the CI/CD pipeline by focusing on PR-based workflow
  • Removed potentially dangerous force pushes
  • Made versioning more explicit with clear commit message tags
  • Improved job organization and naming

This is a good move toward a more controlled release process, which aligns well with GitFlow principles. The changes look good to merge.

One note: There appears to be a small issue in the versioning-minor-increment job - the condition is set to if: "!contains(github.event.head_commit.message, '[ci minor inc]')" which would trigger for commits without the tag. I believe this should be changed to use a positive condition like the patch job does.

@HyperWinX HyperWinX closed this Apr 4, 2025
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.

1 participant

Comments