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.

Sign up for GitHub

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

Jump to bottom

bump-formula-pr: rename @args. #4145

Merged
merged 1 commit into from
May 3, 2018
Merged

bump-formula-pr: rename @args. #4145

merged 1 commit into from
May 3, 2018

Conversation

MikeMcQuaid
Copy link
Member

This shadows the Homebrew variable so let's rename it for now so that --force works again.

@MikeMcQuaid
bump-formula-pr: rename @Args.
c6f21f1 
This shadows the Homebrew variable so let's rename it for now so that
`--force` works again.
GauthamGoli
GauthamGoli approved these changes May 3, 2018
View reviewed changes
Copy link
Contributor

@GauthamGoli GauthamGoli left a comment

Choose a reason for hiding this comment

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

LGTM

@MikeMcQuaid MikeMcQuaid merged commit 62890c6 into Homebrew:master May 3, 2018
@MikeMcQuaid MikeMcQuaid deleted the bump-formula-force branch May 3, 2018 14:19
@lock lock bot added the outdated PR was locked due to age label Jun 2, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Jun 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@GauthamGoli GauthamGoli GauthamGoli approved these changes

@ilovezfs ilovezfs Awaiting requested review from ilovezfs

Assignees
No one assigned
Labels
outdated PR was locked due to age
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MikeMcQuaid @GauthamGoli