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

Add support for publishing homebrew-tap recipe #115

Merged
merged 32 commits into from
Jul 12, 2023

Conversation

Neo2308
Copy link
Member

@Neo2308 Neo2308 commented Jun 5, 2023

Description

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

RadhaKrishna and others added 3 commits June 4, 2023 09:46
* Publish binaries in tar.gz/zip format also
* Configure brew in goreleaser
* Use publisher token for goreleaser
@Neo2308 Neo2308 requested a review from pnickolov as a code owner June 5, 2023 11:53
@codecov-commenter
Copy link

codecov-commenter commented Jun 11, 2023

Codecov Report

Merging #115 (614e0ac) into main (2ead439) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #115   +/-   ##
=======================================
  Coverage   32.08%   32.08%           
=======================================
  Files          27       27           
  Lines        3597     3597           
=======================================
  Hits         1154     1154           
  Misses       2352     2352           
  Partials       91       91           

* Revert to using latest goreleaser
* Update goreleaser version in Makefile to fix ci
* Change goreleaser version in Makefile to v1.18.2 to fix ci
* Revert deprecated key for now.
* Use goreleaser v1.18.2 for now.
@@ -35,9 +35,10 @@ jobs:
- name: Run GoReleaser
uses: goreleaser/goreleaser-action@v3
with:
version: latest
version: 'v1.18.2'
Copy link
Member Author

Choose a reason for hiding this comment

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

Latest version (at the time of writing) is v1.19.2 and it needs extra keys. The changes are incompatible with older versions of goreleaser and goreleaser 1.19.2 requires go 1.20 so the ci breaks. So, for now I have set the goreleaser version to v1.18.2

Will take up the go version update separately and change this back to latest.

@pnickolov pnickolov merged commit d7eff3a into cisco-open:main Jul 12, 2023
7 checks passed
GDW1 pushed a commit to GDW1/fsoc that referenced this pull request Jul 13, 2023
* Add binaries to release archive

* Add support for publishing homebrew-tap recipe

* Publish binaries in tar.gz/zip format also
* Configure brew in goreleaser
* Use publisher token for goreleaser

* Try publishing to experimental repo

* Use separate tokens for publishing binaries and tap

* Fix usage of tap token.

* Fix tap token env variable.

* Raise PR in tap repo.

* Specify branch for PR

* Try publishing to cisco open experimental tap.

* Apply brew test bot suggestion

* Update tap repo name

* Remove tap token

* Revert "Remove tap token"

This reverts commit 3aaf513.

* Fixed deprecated key

* Attempt publish to experimental repo

* Enable debug logging for goreleaser

* Revert go releaser version to v1.18.2

* Change goreleaser flag --verbose -> --debug

* Cleanup

* Remove debug logging from goreleaser
* Change tap repo homebrew-experimental -> homebrew-tap

* Revert to using latest goreleaser

* Revert to using latest goreleaser
* Update goreleaser version in Makefile to fix ci

* Fixing ci

* Change goreleaser version in Makefile to v1.18.2 to fix ci

* Fixing ci

* Revert deprecated key for now.

* Fixing ci

* Use goreleaser v1.18.2 for now.

---------

Co-authored-by: RadhaKrishna <radha.peteti@appdynamics.com>
@Neo2308 Neo2308 deleted the feature/support-publishing-brew-tap branch August 13, 2023 13:12
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