Skip to content

Conversation

@reakaleek
Copy link
Member

@reakaleek reakaleek commented Jan 14, 2025

The following information was also added to the README.md

Release Process

This section outlines the process for releasing a new version of this project.

Versioning

This project uses Semantic Versioning and its version is automatically determined by release-drafter based on the labels of the pull requests merged into the main branch.

See the release-drafter configuration for more details.

Creating a New Release

Every time a pull request is merged into the main branch, release-drafter will
create a draft release or update the existing draft release in the Releases page.

To create a new release you need to publish the existing draft release created by release-drafter.

Important

Make sure the release-drafter workflow is finished before publishing the release.

Note

When a release is published, the create-major-tag workflow
will force push a new major tag in the format vX where X is the major version of the release.
For example, if the release is v1.2.3 was published, the workflow will force push a new tag v1 on the same commit.

@reakaleek reakaleek requested a review from a team January 14, 2025 23:35
@reakaleek reakaleek self-assigned this Jan 14, 2025
@reakaleek
Copy link
Member Author

This will changes the versioning scheme to v{MAJOR}.{MINOR}.{PATCH} (prefixed with v)

Comment on lines 9 to 29
categories:
- title: '💥 Breaking Changes'
labels:
- 'changelog:breaking' # When a breaking change is introduced
- title: '✨ Features'
labels:
- 'changelog:feature' # When a new feature is introduced
- 'changelog:enhancement' # When an existing feature is improved
- title: '🐛 Bug Fixes'
labels:
- 'changelog:fix' # When a bug is fixed
- title: '📝 Documentation'
labels:
- 'changelog:docs' # When documentation is updated
- title: '🧰 Maintenance'
labels:
- 'changelog:chore' # When a chore is done
- 'changelog:ci' # When CI is updated
- 'changelog:dependencies' # When dependencies are updated
exclude-labels:
- 'changelog:skip' # When a PR should be excluded from the changelog
Copy link
Member Author

Choose a reason for hiding this comment

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

We can still discuss the appropriate labels.

Copy link
Member

@Mpdreamz Mpdreamz Jan 15, 2025

Choose a reason for hiding this comment

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

ahh ignore previous comments, can we remove the changelog: prefix from these? Except for the changelog:skip label.

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated labels: e49c1c7

@reakaleek reakaleek mentioned this pull request Jan 14, 2025
@reakaleek
Copy link
Member Author

Existing example: https://github.com/elastic/oblt-actions/releases

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Some nits otherwise lgtm, can't wait to add this!

Comment on lines 9 to 29
categories:
- title: '💥 Breaking Changes'
labels:
- 'changelog:breaking' # When a breaking change is introduced
- title: '✨ Features'
labels:
- 'changelog:feature' # When a new feature is introduced
- 'changelog:enhancement' # When an existing feature is improved
- title: '🐛 Bug Fixes'
labels:
- 'changelog:fix' # When a bug is fixed
- title: '📝 Documentation'
labels:
- 'changelog:docs' # When documentation is updated
- title: '🧰 Maintenance'
labels:
- 'changelog:chore' # When a chore is done
- 'changelog:ci' # When CI is updated
- 'changelog:dependencies' # When dependencies are updated
exclude-labels:
- 'changelog:skip' # When a PR should be excluded from the changelog
Copy link
Member

@Mpdreamz Mpdreamz Jan 15, 2025

Choose a reason for hiding this comment

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

ahh ignore previous comments, can we remove the changelog: prefix from these? Except for the changelog:skip label.

README.md Outdated
> [!NOTE]
> When a release is published, the [create-major-tag workflow](../.github/workflows/create-major-tag.yml)
> will force push a new major tag in the format `vX` where `X` is the major version of the release.
> For example, if the release is `v1.2.3` was published, the workflow will force push a new tag `v1` on the same commit.
Copy link
Member

Choose a reason for hiding this comment

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

Can we drop the v* prefix from the release-drafter? Just because we are already set up with Major.Minor.Patch tags.

Alternatively I can update our minver and workflow triggers.

Copy link
Member Author

@reakaleek reakaleek Jan 15, 2025

Choose a reason for hiding this comment

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

I expected this discussion.

We can use a version with the v prefix.

Would you also like to remove the automation for v1,v2, etc tags? Or do we actually also create this?

this is mainly for the github actions usage convention

Copy link
Member Author

@reakaleek reakaleek Jan 15, 2025

Choose a reason for hiding this comment

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

This will change the versioning scheme to semver without the v prefix.

however, it will still create v[0-9]+ tag.

0fc2d20 and 4e015f3

Copy link
Member

Choose a reason for hiding this comment

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

Thanks! yeah release tags being \d.\d.\d and special v\d tags for github actions works for me!

Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Some nits otherwise lgtm, can't wait to add this!

@reakaleek reakaleek requested a review from Mpdreamz January 15, 2025 08:46
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@reakaleek reakaleek merged commit 1df5344 into main Jan 15, 2025
3 checks passed
@reakaleek reakaleek deleted the feature/release-drafter branch January 15, 2025 09:39
@reakaleek reakaleek added the ci label Jan 15, 2025
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.

3 participants