feat: auto-release abra with goreleaser when a tag is pushed #73

Merged
decentral1se merged 1 commits from knoflook/abra:main into main 2021-09-07 05:15:16 +00:00
Owner

Closes #71

This might change what Drone does when a commit is pushed, because it's now monitoring all branches. So maybe you can decide if it's okay. Most development happens on main so I thought it's not a big deal. Now, if you want to make a release, commit and push your changes and then run:
git tag [version i.e. v0.0.0]
git push origin [version]
and it should automatically build and release.

Closes #71 This might change what Drone does when a commit is pushed, because it's now monitoring all branches. So maybe you can decide if it's okay. Most development happens on main so I thought it's not a big deal. Now, if you want to make a release, commit and push your changes and then run: `git tag [version i.e. v0.0.0]` `git push origin [version]` and it should automatically build and release.
knoflook added the
enhancement
build
labels 2021-09-06 13:49:33 +00:00
decentral1se requested changes 2021-09-06 14:37:48 +00:00
Dismissed
decentral1se left a comment
Owner

💯

💯
.drone.yml Outdated
@ -48,0 +48,4 @@
commands:
- git fetch --tags
depends_on:
- notify on failure
Owner

When a successful build runs, the notify on failure doesn't run because there was no failure.

So, fetch and release will be skipped then. I think you need to do:

depends_on:
- make check
- make build
- make test

You're also missing this?

when:
  event: tag
When a successful build runs, the `notify on failure` doesn't run because there was no failure. So, `fetch` and `release` will be skipped then. I think you need to do: ```yaml depends_on: - make check - make build - make test ``` You're also missing this? ```yaml when: event: tag ```
decentral1se marked this conversation as resolved
.drone.yml Outdated
@ -48,0 +51,4 @@
- notify on failure
- name: release
image: golang
Owner

Can we pin to 1.16 to use the same as development? Using just golang here means latest which might give us inconsistencies somewhere down the line.

Can we pin to `1.16` to use the same as development? Using just `golang` here means `latest` which might give us inconsistencies somewhere down the line.
decentral1se marked this conversation as resolved
go.sum Outdated
@ -37,8 +37,6 @@ cloud.google.com/go/storage v1.5.0/go.mod h1:tpKbwo567HUNpVclU5sGELwQWBDZ8gh0Zeo
cloud.google.com/go/storage v1.6.0/go.mod h1:N7U0C8pVQ/+NIKOBQyamJIeKQKkZ+mxpohlUTyfDhBk=
cloud.google.com/go/storage v1.8.0/go.mod h1:Wv1Oy7z6Yz3DshWRJFhqM/UCfaWIRTdp0RXyy7KQOVs=
cloud.google.com/go/storage v1.10.0/go.mod h1:FLPqc6j+Ki4BU591ie1oL6qBQGu2Bl/tZ9ullr3+Kg0=
coopcloud.tech/tagcmp v0.0.0-20210825131340-4ef558bb7ada h1:MUjktvV/Oc/VCNzJuxMfm+0t9gKfbRrSLoA8sprtZ6U=
Owner

Should this be in this PR? Seems like an unrelated change.

Should this be in this PR? Seems like an unrelated change.
decentral1se marked this conversation as resolved
knoflook force-pushed main from 0e7880e3f4 to bb8124030e 2021-09-06 15:24:14 +00:00 Compare
decentral1se dismissed decentral1se’s review 2021-09-07 05:15:07 +00:00
decentral1se merged commit ff8d9a554a into main 2021-09-07 05:15:16 +00:00
Sign in to join this conversation.
No description provided.