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

Workflow Discussions #41

Open
emptymalei opened this issue Mar 24, 2024 · 7 comments
Open

Workflow Discussions #41

emptymalei opened this issue Mar 24, 2024 · 7 comments
Labels

Comments

@emptymalei
Copy link
Member

emptymalei commented Mar 24, 2024

My current understanding of the workflow.

Working branch

flowchart TB
    working_branch["Working Branch (scope(topic): ...)"]
    main["main Branch"]
    issue["Issue covering the task"]

    review{"PR Review"}
    
    issue -- "link" --> working_branch

    working_branch --> review
    review --> |Approve| main
    review --> |Changes Requested| working_branch
Loading

Release

flowchart TB
    main["main Branch"]
    release["Release Branch (release/x.y.z)\nBump Version + Changelog + [Local]Create/Update Tag by commitizen"]
    tag["Push the Corresponding Tag Version"]
    approved(fa:fa-spinner)

    review{"PR Review"}
    publish["Publish Version"]
    
    release --> review

    review --> |Approve| approved
    approved --> main
    approved --> tag
    review --> |Changes Requested| release

    tag --> publish
Loading

The above workflow requires manual tagging tagging by running some command as we have no automated Actions workflow to create tags. Maybe we automate the tag creation at some point. Then the workflow may become something like release branch merged -> create tag by Actions --> publish by Actions.

@cmp0xff
Copy link
Collaborator

cmp0xff commented Mar 24, 2024

commitizen does create tags automatically as it bumps

@emptymalei
Copy link
Member Author

emptymalei commented Mar 24, 2024

commitizen does create tags automatically as it bumps

yep, that is also what the workflow suggests, that we don't create tags by ourselves.

@emptymalei
Copy link
Member Author

emptymalei commented Mar 24, 2024

So the idea is that, as long as we don't git tag -a, this workflow seems to be fine and we don't need to worry about tags.
But it is much better to have an automation to auto create tags at some point.

@cmp0xff
Copy link
Collaborator

cmp0xff commented Mar 24, 2024

commitizen does create tags automatically as it bumps

yep, that is also what the workflow suggests, that we don't create tags by ourselves.

I was confused when I read the statement

The above workflow requires manual tagging as we have no automated tag creation.

My understanding is that commitizen creates tags automatically. The only manual things here are

  1. creating a release branch
  2. run cz bump

And I do not think we can essentially automate this further, either..

@emptymalei
Copy link
Member Author

emptymalei commented Mar 24, 2024

commitizen does create tags automatically as it bumps

yep, that is also what the workflow suggests, that we don't create tags by ourselves.

I was confused when I read the statement

The above workflow requires manual tagging as we have no automated tag creation.

My understanding is that commitizen creates tags automatically. The only manual things here are

  1. creating a release branch
  2. run cz bump

And I do not think we can essentially automate this further, either..

Ah, by manually tagging I mean we don't use Actions to automatically tags the commit. Manually means we have to run command by hand.

Edit:
I updated the statement a bit.

@emptymalei
Copy link
Member Author

Not sure what is a good solution to this: #33 (comment)

I guess the first thing to do is to protected the tags.

Screenshot 2024-03-24 at 11 57 34

With this we limit who can push tags to remote.

@emptymalei
Copy link
Member Author

Cross reference
#42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants