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

Trigger Release with tag 'releaseTrigger' #256

Merged
merged 8 commits into from May 19, 2020

Conversation

aepfli
Copy link
Member

@aepfli aepfli commented May 2, 2020

This changes will trigger a build on master,
when there is a tag 'releaseTrigger' created.

It will notify our normal GitHub Actions, and
retrigger a build. The payload will tell the
our normal build, that this is a release, and
it will trigger the build.

Downside, this will only work in MASTER for now.
If we want to support other branches, we need
to use our combined brain power, how to retrieve
a branch name from a git tag. It is not such
a trivial problem.

Questions:

  1. is it okay that we can only release on master?
  2. does this work for everyone with the tagging?
  3. shall we move the deletion of the tag also into the triggering?

I hereby agree to the terms of the JUnit Pioneer Contributor License Agreement.

Simon Schrottner added 2 commits May 2, 2020 23:08
This changes will trigger a build on master,
when there is a tag 'releaseTrigger' created.

It will notify our normal GitHub Actions, and
retrigger a build. The payload will tell the
our normal build, that this is a release, and
it will trigger the build.

Downside, this will only work in MASTER for now.
If we want to support other branches, we need
to use our combined brain power, how to retrieve
a branch name from a git tag. It is not such
a trivial problem.
@Bukama
Copy link
Member

Bukama commented May 3, 2020

Downside, this will only work in MASTER for now.
If we want to support other branches, we need
to use our combined brain power, how to retrieve
a branch name from a git tag. It is not such
a trivial problem.

Tags are not related to branches, but to commits.

Questions:

  1. is it okay that we can only release on master?

Sure, same as of now

  1. does this work for everyone with the tagging?

That made me think - can everyone create a tag and start a release? Or only maintainers, who have write permissions on the repo?

  1. shall we move the deletion of the tag also into the triggering?

In my opinion it should be the last step of the process, so when the release was successful the tag is gone.

Simon Schrottner added 2 commits May 3, 2020 12:29
This changes will trigger a build on master,
when there is a tag 'releaseTrigger' created.

It will notify our normal GitHub Actions, and
retrigger a build. The payload will tell the
our normal build, that this is a release, and
it will trigger the build.

Downside, this will only work in MASTER for now.
If we want to support other branches, we need
to use our combined brain power, how to retrieve
a branch name from a git tag. It is not such
a trivial problem.
…gAction

# Conflicts:
#	.github/workflows/actions.yml
#	.github/workflows/releaseTrigger.yml
@aepfli aepfli requested a review from beatngu13 May 3, 2020 13:22
@aepfli
Copy link
Member Author

aepfli commented May 3, 2020

@beatngu13 as you are also deeper into this build pipeline issues, what do you think about this approach, splitting deployment into own yaml and waiting of all other suites which are github actions, to finish?

@aepfli aepfli linked an issue May 3, 2020 that may be closed by this pull request
.github/workflows/releaseTrigger.yml Outdated Show resolved Hide resolved
.github/workflows/releaseTrigger.yml Show resolved Hide resolved
.github/workflows/releaseTrigger.yml Outdated Show resolved Hide resolved
Copy link
Member

@beatngu13 beatngu13 left a comment

Choose a reason for hiding this comment

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

@beatngu13 as you are also deeper into this build pipeline issues, what do you think about this approach, splitting deployment into own yaml and waiting of all other suites which are github actions, to finish?

I haven't work with some of these actions, so I can't tell how they work in practice. But overall I like the approach very much and it appears to be a great fit for the targeted workflow. 👍

.github/workflows/releaseTrigger.yml Outdated Show resolved Hide resolved
.github/workflows/releaseTrigger.yml Outdated Show resolved Hide resolved
.github/workflows/releaseTrigger.yml Outdated Show resolved Hide resolved
.github/workflows/releaseTrigger.yml Show resolved Hide resolved
Co-authored-by: Daniel Kraus <daniel.kraus@mailbox.org>
@aepfli aepfli marked this pull request as ready for review May 11, 2020 12:38
@sonarcloud
Copy link

sonarcloud bot commented May 19, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@nipafx nipafx merged commit 80b3e45 into junit-pioneer:master May 19, 2020
@aepfli aepfli deleted the issues/TagAction branch May 19, 2020 19:13
nipafx pushed a commit that referenced this pull request Feb 28, 2021
Migrate to new Shipkit plugins (#410 / #419)

Shipkit is deprecated and the GitHub project[1] is archived. They
write:

> "One Shipkit Gradle plugin to rule them all" approach has proven
> hard to maintain for the team. We [have] converted Shipkit into a
> narrow set of small libraries (design note[2]). Several customers,
> inclu[d]ing Mockito project have already migrated.

More preecisely, Shipkit was split into these projects:

* shipkit-auto-version[3]
* shipkit-changelog[4]
* shipkit-github-release (no source on GitHub?)

This change updates the build accordingly.

Closes: #410
Maybe solves: #256 (can't be determined without a release build)
PR: #419

[1]: https://github.com/mockito/shipkit
[2]: https://github.com/mockito/shipkit/blob/master/docs/design-specs/future-shipkit.md
[3]: https://github.com/shipkit/shipkit-auto-version
[4]: https://github.com/shipkit/shipkit-changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Evaluate ShipKit
4 participants