[quote=“Weyrling, post:4, topic:8877”]I’m fairly new to github but the basic process looks something like this:
First off you’ll want to go to the github: https://github.com/CleverRaven/Cataclysm-DDA
Step two would be to make an account or login.
Step three is to make your own fork, right below the Sign In button is one labeled Fork which makes a copy that’s under your own account.
You would then modify this new fork of the game with your changes.
Afterwards you go back into CleverRaven’s github (linked above) and go to Pull Requests on the right side, make a new one and find your fork in the comparison list and then submit the PR.
If I’m horribly wrong, somebody will probably correct me pretty soon.[/quote]
Not so horribly. Our write-up on contributing might be helpful: CONTRIBUTING.md.
Vital step that can make the difference between me merging you, cuing pyros, welcome to the open-source community, mention in Latest experimental features., warm fuzzies, etc OR the Baleful Frown, “KA101 removed their assignment”, FAIL : clone your web-based repo to your local computer, make the new branch and the changes there, TEST THEM ON YOUR END, and then commit/upload/PR.
JSON PRs such as this are fairly easy to write up (balancing, not so much, especially something like the LMOE which can come loaded to the gills, barren, or in between–but that’s a separate post), and also easy to screw up the syntax and formatting.
Exclusive use of the web interface is notorious for spacing things Weirdly and making the code tougher to read, and chances are you didn’t actually put your web-only PR through its paces, since it was never actually resident in your DDA.
Actually put your PR through what it’s supposed to do (start a game and chose the profession, in this case). If the parser stops you with Angry Pink Text, read and fix.
Folks who don’t do this sometimes submit PRs that are full of JSON errors, which wastes dev time when we’re checking you for merge. Wasting our time is not optimal.