Skip to content

run "brew style --fix awless.rb" - #5

Open
maelvls wants to merge 1 commit into
wallix:masterfrom
maelvls:fix-bottle-syntax
Open

maelvls wants to merge 1 commit into
wallix:masterfrom
maelvls:fix-bottle-syntax

Conversation

@maelvls

@maelvls maelvls commented Apr 15, 2021

Copy link
Copy Markdown

These warnings were showing up:

Warning: Calling cellar in a bottle block is deprecated! Use brew style --fix on the formula to update the style or use sha256 with a
cellar: argument instead. Please report this issue to the
wallix/awless tap (not Homebrew/brew or Homebrew/core), or even better,
submit a PR to fix it.

Warning: Calling sha256 "digest" => :tag in a bottle block is
deprecated! Use brew style --fix on the formula to update the style or
use sha256 tag: "digest" instead. Please report this issue to the
wallix/awless tap (not Homebrew/brew or Homebrew/core), or even better,
submit a PR to fix it.

So I figured I would run brew style --fix awless.rb and submit a PR 😁

These warnings were showing up:

Warning: Calling `cellar` in a bottle block is deprecated! Use `brew
style --fix` on the formula to update the style or use `sha256` with a
`cellar:` argument instead.  Please report this issue to the
wallix/awless tap (not Homebrew/brew or Homebrew/core), or even better,
submit a PR to fix it.

Warning: Calling `sha256 "digest" => :tag` in a bottle block is
deprecated! Use `brew style --fix` on the formula to update the style or
use `sha256 tag: "digest"` instead.  Please report this issue to the
wallix/awless tap (not Homebrew/brew or Homebrew/core), or even better,
submit a PR to fix it.
@pabloxio

pabloxio commented Jun 2, 2021

Copy link
Copy Markdown

Is the awless project deprecated at this point?

@luisdavim luisdavim left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@luisdavim

Copy link
Copy Markdown

Can someone merge this?

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.

3 participants