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

feature(ci): release gemini using goreleaser and github actions #427

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

CodeLieutenant
Copy link
Contributor

@CodeLieutenant CodeLieutenant commented Sep 16, 2024

In order to make use of goreleaser, we need GitHub Action to build and gemini and generate changelog

This is the action to do so.

Commented out blob

Inside the repository, we don't have any AWS Keys to upload gemini binaries to S3 Bucket. until that gets sorted out, it is better to leave it commented and have the gemini release as soon as possible to test the #424 in SCT.

@CodeLieutenant CodeLieutenant self-assigned this Sep 16, 2024
@CodeLieutenant CodeLieutenant added enhancement New feature or request good first issue Good for newcomers labels Sep 16, 2024
.goreleaser.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@fruch fruch left a comment

Choose a reason for hiding this comment

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

LGTM

name: goreleaser

on:
push:
Copy link
Collaborator

Choose a reason for hiding this comment

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

When do you want it to be triggered ? Currently it is going to be run on every push for every branch.

We can trigger it when:

  1. Maintainer calls it via actions tab
on:
  workflow_dispatch
  1. Make it triggered automagically if PR have label is being merged:
on:
  pull_request:
    branches: 
      - main
    types: [closed]

jobs:
  my-action:
    if: ${{ github.event.pull_request.merged }}

Copy link
Collaborator

Choose a reason for hiding this comment

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

that's not correct, read the next line, it's limited to tag pushes only

Copy link
Collaborator

Choose a reason for hiding this comment

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

we might want to limit this to the master branch.

but this workflow, won't be running on PRs

@fruch fruch merged commit ab3e873 into scylladb:master Sep 17, 2024
3 of 5 checks passed
@CodeLieutenant CodeLieutenant deleted the feat/gh-actions-release branch September 17, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants