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

bat: install bundled fish completions #40280

Closed
wants to merge 2 commits into from

Conversation

lwolfsonkin
Copy link
Contributor

@lwolfsonkin lwolfsonkin commented May 24, 2019

Installs bundled fish completions for bat.

In bat version v0.11.0, they added manually written fish completions (in sharkdp/bat#524 and sharkdp/bat#555) since they disabled the automatically generated completions in v0.10.0. This update simply installs them during brew install. Note that this is a sort of soft reversion of #36116.


  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?
    • Unfortunately, my Homebrew errors out when I try to run brew audit like in freetds 1.00.15 #5561, so I could not. I attempted troubleshooting, but to no avail.

@lwolfsonkin lwolfsonkin changed the title Installs bundled fish completions for [bat](https://github.com/sharkdp/bat). In bat version [v0.11.0](https://github.com/sharkdp/bat/releases/tag/v0.11.0), they added [manually written](https://github.com/sharkdp/bat/pull/524) [fish completions](https://github.com/sharkdp/bat/pull/555) since they disabled the automatically generated completions in [v0.10.0](https://github.com/sharkdp/bat/releases/tag/v0.10.0). This update simply installs them during brew install. Update bat to install bundled fish completions May 24, 2019
@lwolfsonkin lwolfsonkin changed the title Update bat to install bundled fish completions 🦇 install bundled fish completions May 24, 2019
@lwolfsonkin lwolfsonkin changed the title 🦇 install bundled fish completions bat: install bundled fish completions May 24, 2019
Copy link
Contributor

@zbeekman zbeekman left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@zbeekman zbeekman left a comment

Choose a reason for hiding this comment

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

I spoke too soon. If you want this change to be shipped out to users, can you please add a revision bump? i.e., revision 1 below the sha256 checksum? You can automate the process with brew bump-revision bat.

Thanks!

@lwolfsonkin
Copy link
Contributor Author

Revision bumped!

Thanks for the review, @zbeekman!

Copy link
Contributor

@zbeekman zbeekman left a comment

Choose a reason for hiding this comment

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

LGTM

@zbeekman
Copy link
Contributor

@lwolfsonkin For future reference: Please set the commit message to start with the formula name (like the PR title) and squash commits.

Thanks!

@zbeekman
Copy link
Contributor

Thanks for your contribution to Homebrew! 🎉 🥇

Without awesome contributors like you, it would be impossible to maintain Homebrew to the high level of quality users have come to expect. Thank you!!!!

@zbeekman zbeekman closed this May 25, 2019
zbeekman pushed a commit that referenced this pull request May 25, 2019
@lwolfsonkin
Copy link
Contributor Author

Will squash for the future! Sorry about that.

And no problem! :)

zgracem added a commit to zgracem/dotconfig that referenced this pull request Jun 4, 2019
These are included w/ installation as of
Homebrew/homebrew-core#40280
@lock lock bot added the outdated PR was locked due to age label Feb 14, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Feb 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants