07-21-2016, 03:06 PM
(This post was last modified: 07-23-2016, 08:37 PM by Mordent. Edited 1 time in total.
Edit Reason: Updating link to correct base branch
)
I didn't get any feedback for the idea in the Ideas & Suggestions thread, but I figured I'd just go ahead and make a patch.
The Readme thread seems a little out of date based on the others I'm reading (i.e. they don't create PRs to the main repo, instead including the changes for you lot to do with as you see fit), so I've given this a shot and let me know if the actual process is off; I've got some other small easy fixes I'd like to have a crack at in the near future so getting the process down now would be great.
Effect(s)
Testing
Code
The Readme thread seems a little out of date based on the others I'm reading (i.e. they don't create PRs to the main repo, instead including the changes for you lot to do with as you see fit), so I've given this a shot and let me know if the actual process is off; I've got some other small easy fixes I'd like to have a crack at in the near future so getting the process down now would be great.
Effect(s)
- Adds the hemostat to the medical and robotics fabricators, costing 1 metal (same as the scalpel, circular saw, etc.).
Testing
- I tested on my local build (my changes on top of the goonstation-2016-byond510 branch) and it seems fine.
Code