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

feat: release please setup to automate releases #308

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

iamvigneshwars
Copy link
Contributor

No description provided.

@iamvigneshwars iamvigneshwars added enhancement New feature or request github actions Changes to CI/CD labels Nov 29, 2024
@iamvigneshwars iamvigneshwars self-assigned this Nov 29, 2024
Copy link
Member

@garryod garryod left a comment

Choose a reason for hiding this comment

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

A few thoughts:

  • How do we want to handle the helm linting which at present requires us to bump the chart version with each change - presumably this should only now be bumped by release-please?
  • This will still publish containers for every tag, which we don't want
  • Is there a a reason we separate the docs? And what if the mkdocs.yaml is updated?
  • How do we ensure that commits are conventional? As non-conventional ones will break our usage of release-please

@iamvigneshwars iamvigneshwars force-pushed the release-please branch 4 times, most recently from 8da6d8a to efa616f Compare December 2, 2024 13:16
@iamvigneshwars iamvigneshwars changed the title feat: Release please setup to automate releases feat: release please setup to automate releases Dec 2, 2024
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/_sessionspaces_container.yaml Outdated Show resolved Hide resolved
.github/workflows/_graph_proxy_container.yaml Outdated Show resolved Hide resolved
.github/workflows/_commit_lint.yaml Outdated Show resolved Hide resolved
.github/workflows/_commit_lint.yaml Outdated Show resolved Hide resolved
@iamvigneshwars iamvigneshwars force-pushed the release-please branch 3 times, most recently from e455363 to ea7d97c Compare December 4, 2024 15:57
.github/workflows/_commit_lint.yaml Show resolved Hide resolved
.github/workflows/_commit_lint.yaml Outdated Show resolved Hide resolved
.github/workflows/_commit_lint.yaml Outdated Show resolved Hide resolved
.github/workflows/_commit_lint.yaml Outdated Show resolved Hide resolved
.github/workflows/_dashboard_container.yaml Outdated Show resolved Hide resolved
@iamvigneshwars iamvigneshwars force-pushed the release-please branch 3 times, most recently from 2f08242 to ac5b9f0 Compare December 5, 2024 16:58
@iamvigneshwars iamvigneshwars force-pushed the release-please branch 3 times, most recently from ef6c299 to b5596bd Compare December 5, 2024 18:38
@daurer
Copy link
Contributor

daurer commented Dec 11, 2024

We should probably modify the commit messages in the dependabot to conventional commits. See here how to configure dependabot. .

This issue could also be relevant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request github actions Changes to CI/CD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants