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

Remove Brewfile.lock.json generation. #1509

Merged
merged 1 commit into from
Nov 27, 2024
Merged

Remove Brewfile.lock.json generation. #1509

merged 1 commit into from
Nov 27, 2024

Conversation

MikeMcQuaid
Copy link
Member

This was originally added to faciliate later functionality from other maintainers that never materialized.

I've never heard anyone say this was useful and a lot of people have complained that it's useless or annoying.

I won't merge this immediately to give people a change to comment.

This was originally added to faciliate later functionality from other
maintainers that never materialized.

I've never heard anyone say this was useful and a lot of people have
complained that it's useless or annoying.
Copy link
Member

@p-linnane p-linnane left a comment

Choose a reason for hiding this comment

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

Makes sense. I've also never understood the purpose, so your explanation makes sense.

@colindean
Copy link
Member

I have no objections. I've got a line for Brewfile.lock.json in every .gitignore in a repo with a Brewfile.

@jacobbednarz
Copy link
Member

works for me!

@p-linnane p-linnane merged commit 1419e23 into master Nov 27, 2024
5 checks passed
@p-linnane p-linnane deleted the remove_lockfile branch November 27, 2024 00:48
marcusandre added a commit to marcusandre/dotfiles that referenced this pull request Nov 29, 2024
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.

4 participants