Skip to content

Conversation

@dexhorthy
Copy link
Contributor

@dexhorthy dexhorthy commented Aug 10, 2020

Pulling some of the logic from the Makefile into the CLI here.

No special support for GitHub actions yet, but this shouldn't break any situation in which folks are using the Makefile and its special GHA logic to release.


$ go run ./cli release create --auto
  • Reading Environment ✓

Prepared to create release with defaults:

    yaml-dir        "./manifests"
    promote         "auto-release"
    version         "auto-release-180c2e-dirty"
    release-notes   "CLI release by dex on 11 Aug 20 15:44 CDT"
    ensure-channel  true

Create release with these properties? [Y/n] █

  • Reading manifests from ./manifests ✓
  • Creating Release ✓
    • SEQUENCE: 128
  • Promoting ✓
    • Channel 1fy7ktlXka3WylbLIR3w218CO1G successfully set to release 128

@dexhorthy dexhorthy force-pushed the auto-release branch 5 times, most recently from 4853172 to 8dc9c04 Compare August 11, 2020 20:45
@dexhorthy dexhorthy changed the title auto-generating release properties fixing brew warning in goreleaser, auto-generating release properties, move build to GHA Aug 11, 2020
tags:
- 'v*'

jobs:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I haven't tested this but will try a release in a bit and see if it works, may have a followup PR

…, move build to GHA

Pulling some of the logic from the Makefile into the CLI here.

No special support for GitHub actions yet, but this shouldn't break any situation in which folks are using the Makefile and its special GHA logic to release.

```

$ go run ./cli release create --auto
  • Reading Environment ✓

Prepared to create release with defaults:

    yaml-dir        "./manifests"
    promote         "auto-release"
    version         "auto-release-180c2e-dirty"
    release-notes   "CLI release by dex on 11 Aug 20 15:44 CDT"
    ensure-channel  true

Create release with these properties? [Y/n] █

  • Reading manifests from ./manifests ✓
  • Creating Release ✓
    • SEQUENCE: 128
  • Promoting ✓
    • Channel 1fy7ktlXka3WylbLIR3w218CO1G successfully set to release 128

```
@dexhorthy dexhorthy merged commit 57bbd81 into master Aug 13, 2020
@dexhorthy dexhorthy deleted the auto-release branch August 24, 2020 14:57
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.

3 participants