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

only brass recipe ingame now uses proper protective equipment & the proper proficiency #64398

Merged
merged 4 commits into from
Mar 27, 2023

Conversation

Daved27hundred
Copy link
Contributor

Summary

None

Purpose of change

Making/melting brass without a respirator of some kind is dangerous due to the fumes from the zinc causing health problems. Also, it uses the Blacksmithing proficiency despite not having any steel or iron in it's recipe. This pr fixes that.

Describe the solution

added the toolset ppe_gas from PR #62067 to the recipe as well as changed prof_blacksmithing to prof_redsmithing.

Describe alternatives you've considered

Removing the whole recipe, maybe?

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Mar 19, 2023
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Mar 26, 2023
@dseguin dseguin merged commit 791b6d1 into CleverRaven:master Mar 27, 2023
@Daved27hundred Daved27hundred deleted the mmmm-zinc-fumes-yummy branch May 9, 2023 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants