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

feat: automate brew formula update #709

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

krushnarout
Copy link
Contributor


name: feat: automate brew formula update
about: automate brew formula update
title: "feat: automate brew formula update"
labels: ''
assignees: ''


description

automate brew formula update using brew bump-formula-pr

Closes #692
/claim #692

type of change

  • new feature

checklist

  • MOST IMPORTANT: this PR will require less than 30 min to review, merge, and release to production and not crash in the hand of thousands of users
  • i have read the CONTRIBUTING.md file
  • i have updated the documentation if necessary
  • my changes generate no new warnings
  • i have added tests that prove my fix is effective or that my feature works

additional notes

any other relevant information about the pr.

Copy link

vercel bot commented Nov 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
screenpipe ❌ Failed (Inspect) Nov 20, 2024 3:03pm

@louis030195
Copy link
Collaborator

cool, great work!

how can i test this to make sure it works?

@louis030195
Copy link
Collaborator

https://github.com/Homebrew/homebrew-core/pull/196345/files

@m13v any chance you can add the ui monitoring lib there? (maybe after this PR is merged)

think we need to send PR to brew official to add it

@krushnarout
Copy link
Contributor Author

cool, great work!

how can i test this to make sure it works?

Thank you!

Use --dry-run to test this locally without creating PR.

tmp_dir=$(mktemp -d)
cd "$tmp_dir"

export VERSION="0.1.99"
export GITHUB_REPOSITORY="mediar-ai/screenpipe"
export MATRIX_TARGET="x86_64-apple-darwin"

curl -LO "https://github.com/${GITHUB_REPOSITORY}/releases/download/v${VERSION}/screenpipe-${VERSION}-${MATRIX_TARGET}.tar.gz"

export SHA=$(shasum -a 256 screenpipe-*.tar.gz | cut -d ' ' -f 1)
export URL="https://github.com/${GITHUB_REPOSITORY}/releases/download/v${VERSION}/screenpipe-${VERSION}-${MATRIX_TARGET}.tar.gz"

# Added --dry-run to test without creating PR
brew bump-formula-pr screenpipe \
  --version=${VERSION} \
  --url="${URL}" \
  --sha256="${SHA}" \
  --message="Test: Update screenpipe to version ${VERSION} for ${MATRIX_TARGET}" \
  --no-browse \
  --no-audit \
  --dry-run
  
  cd -
rm -rf "$tmp_dir"
Screen.Recording.2024-11-21.at.7.51.07.AM.mov

@louis030195
Copy link
Collaborator

/approve
thanks!

Copy link

algora-pbc bot commented Nov 21, 2024

@louis030195: The claim has been successfully added to reward-all. You can visit your dashboard to complete the payment.

@louis030195 louis030195 merged commit c6eacf5 into mediar-ai:main Nov 21, 2024
2 of 7 checks passed
@louis030195
Copy link
Collaborator

@krushnarout do you have any idea how to make this work for updating the linux brew too?

@krushnarout
Copy link
Contributor Author

@krushnarout do you have any idea how to make this work for updating the linux brew too?

@louis030195 I’m not sure how to make it work for updating Linux brew at the moment. There's limited information available on this, but if I had more time to look into it, I might be able to find a solution.

@louis030195
Copy link
Collaborator

@louis030195
Copy link
Collaborator

Homebrew/homebrew-core#198524 broken 😕

@krushnarout
Copy link
Contributor Author

Homebrew/homebrew-core#198524 broken 😕

Hi @louis030195! I noticed you haven't released a new version yet, but you're trying to update the formula without one.

@louis030195
Copy link
Collaborator

louis030195 commented Nov 22, 2024

@krushnarout what do you mean?

release-cli usually required:

git tag v<my version>
git push origin v<my version>

and it would release to brew end-to-end

now it's just broken

i released 3 times, lost 1h of my day and it does not work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bounty] automating official brew update
2 participants