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

Upgrade footloose to 0.6.3 #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

cdenneen
Copy link

@cdenneen cdenneen commented Apr 3, 2020

Upgrade footloose version to 0.6.3

Upgrade footloose version to 0.6.3
Copy link
Contributor

@dholbach dholbach left a comment

Choose a reason for hiding this comment

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

This generally looks good to me, just not sure why the file says it's auto-generated, but it apparently was not updated for the releases... @stefanprodan Do you have an idea?

c.f. https://github.com/weaveworks/homebrew-tap/blob/master/Formula/footloose.rb#L1

@kke
Copy link

kke commented May 13, 2020

Ran into this.

Also there's no --HEAD which would enable to just install the latest master if you so wish.

@kingdonb
Copy link
Contributor

I'm looking into this. I noticed the auto-generated notices in the gitops formula and also see that it is one version behind.

It says that goreleaser should publish these releases, but in the weave-gitops repository I don't find any mention of that.

I'm assuming that someone has been capturing the output of the homebrew tap updates somehow and putting them in manually. @J-Thompson12 Can you clue me into how this works, or how it is supposed to work anyway? (Your name is on the last commit that updated (1487b7d) the GitOps CLI.)

@kingdonb kingdonb self-assigned this Feb 25, 2022
@J-Thompson12
Copy link
Contributor

@kingdonb yeah so we have had problems with weave-gitops and pushing to this repo. So I was adding the changes manually for a bit but that is no longer a thing.

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.

6 participants