-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Machete proliferation #76176
Machete proliferation #76176
Conversation
What the hell? |
Basic build test was being stupid. |
Uhh, I'm guessing you PR'd this from the same branch you used to make the FBI profession? Regarding the 90 empty commits |
Why do you keep randomly failing the tests? |
I can honestly say I've never encountered a gardening machete in the US before, but that's fine. |
Is there any chance you can clean up the commit stack to remove all the extraneous ones? |
I have no idea how, I tried purging my master file and it didn't work. I will avoid using master after this, but I don't have any recourse to fix it in the moment. Luckily it only alters the one file, so it should be good to go |
With your permission I can try to fix it myself. (I mean, I have permission, it's a checkbox when you open the PR, but it's not usually used for this). |
That would be lovely! |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Summary
Balance "Add the machete to relevant lootgroups."
Purpose of change
I was scrolling through the HHG, and realized that the common household machete, most often used for brush clearing and landscaping, was absurdly rare. This PR adds it to the relevant lootgroups.
Describe the solution
Adds it to the gardening tools, hand tools, and household tools loot lists.
Describe alternatives you've considered
not doing that
Testing
Additional context
I don't know why it says there are 90 commits included, all I know is that this only changes 1 file.