docs: add cask maintainer guide
This commit is contained in:
parent
526c06da0c
commit
0f47290622
59
docs/Cask-Maintainer-Guide.md
Normal file
59
docs/Cask-Maintainer-Guide.md
Normal file
@ -0,0 +1,59 @@
|
|||||||
|
# Cask Maintainer Guide
|
||||||
|
|
||||||
|
This guide is intended to help maintainers effectively maintain the cask repositories.
|
||||||
|
It is meant to be used in conjunction with the more generic [Maintainer Guidelines](Maintainer-Guidelines.md).
|
||||||
|
|
||||||
|
This guide applies to all four of the cask repositories:
|
||||||
|
|
||||||
|
- [Homebrew/homebrew-cask](https://github.com/Homebrew/homebrew-cask): The main cask repository
|
||||||
|
- [Homebrew/homebrew-cask-drivers](https://github.com/Homebrew/homebrew-cask-drivers): Casks of drivers
|
||||||
|
- [Homebrew/homebrew-cask-fonts](https://github.com/Homebrew/homebrew-cask-fonts): Casks of fonts
|
||||||
|
- [Homebrew/homebrew-cask-versions](https://github.com/Homebrew/homebrew-cask-versions): Alternate versions of Casks
|
||||||
|
|
||||||
|
## Common Situations
|
||||||
|
|
||||||
|
Here is a list of the most common situations that arise in PRs and how to handle them:
|
||||||
|
|
||||||
|
- The `version` and `sha256` both change (keeping the same format): Merge
|
||||||
|
- Only the `sha256` changes: Merge unless the version needs to be updated as well.
|
||||||
|
It’s not uncommon for upstream vendors to update versions in-place.
|
||||||
|
- `livecheck` is updated: Use your best judgement and try to make sure that the changes
|
||||||
|
follow the [`livecheck` guidelines](Brew-Livecheck.md).
|
||||||
|
- Only the `version` changes or the `version` format changes: Use your best judgement and
|
||||||
|
merge if it seems correct (this is relatively rare).
|
||||||
|
- Other changes (including adding new casks): Use the [Cask Cookbook](Cask-Cookbook.md) to determine what's correct.
|
||||||
|
|
||||||
|
If in doubt, ask another cask maintainer on GitHub or Slack.
|
||||||
|
|
||||||
|
Note that unlike in formulae, casks do not consider the `sha256` stanza as meaningful security measure as maintainers cannot realistically check them for authenticity. Casks download from upstream; if a malicious actor compromised a URL, they could just as well compromise a version and make it look like an update.
|
||||||
|
|
||||||
|
## Merging
|
||||||
|
|
||||||
|
### Approvals
|
||||||
|
|
||||||
|
In general, PRs in the cask repositories should have at least one approval from a maintainer
|
||||||
|
before being merged.
|
||||||
|
|
||||||
|
If desired, a maintainer can self-approve one of their PRs using the
|
||||||
|
`self-approve` GitHub Actions workflow to satisfy this requirement. To trigger a self-approval, navigate to the
|
||||||
|
["Self-approve a Pull Request" section of the Actions tab](https://github.com/Homebrew/homebrew-cask/actions/workflows/self-approve.yml),
|
||||||
|
click on "Run workflow", enter the PR number and click "Run workflow".
|
||||||
|
|
||||||
|
### Merge Types
|
||||||
|
|
||||||
|
In general, using GitHub's Squash and Merge button is the best way to merge a PR. This can be used when
|
||||||
|
the PR modifies only one cask, regardless of the number of commits or whether the commit message
|
||||||
|
format is correct. When merging using this method, the commit message can be modified if needed.
|
||||||
|
Usually, version bump commit messages follow the form `Update CASK from OLD_VERSION to NEW_VERSION`.
|
||||||
|
|
||||||
|
If the PR modifies multiple casks, use the Rebase and Merge button to merge the PR. This will use the
|
||||||
|
commit messages from the PR, so make sure that they are appropriate before merging. If needed,
|
||||||
|
checkout the PR, squash/reword the commits and force-push back to the PR branch to ensure the proper commit format.
|
||||||
|
|
||||||
|
Finally, make sure to thank the contributor for submitting a PR!
|
||||||
|
|
||||||
|
## Other Tips
|
||||||
|
|
||||||
|
A maintainer can easily rebase a PR onto the latest `master` branch by adding a `/rebase` comment.
|
||||||
|
`BrewTestBot` will automatically rebase the PR and add a reaction to
|
||||||
|
the comment once the rebase is in progress and complete.
|
@ -66,6 +66,7 @@
|
|||||||
- [Releases](Releases.md)
|
- [Releases](Releases.md)
|
||||||
- [Developer/Internal API Documentation](https://rubydoc.brew.sh)
|
- [Developer/Internal API Documentation](https://rubydoc.brew.sh)
|
||||||
- [Homebrew/homebrew-core merge checklist](Homebrew-homebrew-core-Merge-Checklist.md)
|
- [Homebrew/homebrew-core merge checklist](Homebrew-homebrew-core-Merge-Checklist.md)
|
||||||
|
- [Cask Maintainer Guide](Cask-Maintainer-Guide.md)
|
||||||
|
|
||||||
## Governance
|
## Governance
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user