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

treesheets 8473991279, with source switched to GitHub Releases #170193

Merged
merged 3 commits into from Apr 2, 2024

Conversation

danyalaytekin
Copy link
Contributor

@danyalaytekin danyalaytekin commented Mar 29, 2024

Changes

  • update treesheets to version 8473991279
  • fetch from GitHub Releases, which is now the source used by the download link on the TreeSheets website and in its GitHub readme
  • add zap trash harvested from AppCleaner anecdote
  • add uninstall quit id

Checks

Checks for 'edit cask'

Relevant checks from 'new cask' section

  • brew uninstall --cask <cask> worked successfully.

@danyalaytekin
Copy link
Contributor Author

danyalaytekin commented Mar 29, 2024

(Done) Sorry, I missed the --online. I'm looking into the regex or whatever it'll turn out to be.

  • Update and test livecheck pattern

@danyalaytekin danyalaytekin changed the title treesheets 8473991279, with source switched to GitHub treesheets 8473991279, with source switched to GitHub Releases Mar 30, 2024
@razvanazamfirei
Copy link
Member

razvanazamfirei commented Apr 1, 2024

This looks good and thank you for opening this! Just bumping the version. I verified the homepage links to the Github releases.

@razvanazamfirei razvanazamfirei added the ready to merge PR can be merged once CI is green label Apr 1, 2024
@krehel krehel merged commit 35dd763 into Homebrew:master Apr 2, 2024
11 checks passed
@danyalaytekin danyalaytekin deleted the treesheets-from-github branch April 3, 2024 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge PR can be merged once CI is green
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants