Want to contribute to Socialhome? Great! <3 Please read on for some guidelines.
First things first¶
Please make sure you have some knowledge of what the software is for before jumping in to write code. You don’t even have to install a development environment. Just head to https://socialhome.network, create an account and play around.
If you already are a user or run your own instance, you probably have some ideas on how to contribute already. The best contributions come from real personal need.
Finding things to do¶
We have an issue tracker on GitHub. If you don’t already have an idea on what to do, check out the issues listed there. Some issues are labeled as newcomer. These are easy picking tasks for those either new to Socialhome or with less knowledge of Django.
See also our Architecture for high level plans.
Contributions are not just code. Please feel free to log not only bugs but also enhancement ideas in the issue tracker. For issues that have not been confirmed (= they don’t have the label “ready”), triaging is important contribution also. Reproducing and supplying more information on these issues is valuable contributon to the project.
So you’re ready to write code! Great! Please remember though that the project already has a vision, the software has architecture and the project maintainer will have strong opinions on how things should be implemented. Before you write a lot of code that even remotely feels like it would need a design decision, please always discuss your plan first with the project maintainer. Otherwise you might spend a lot of time writing code only to be told the code will not be merged because it doesn’t fit into the grand plan.
Please don’t be afraid to get in touch, see channels in the Community pages.
As a general rule all code must have unit tests. For bug fixes provide a test that ensures the bug will not be back and for features always add a good enough coverage. PR’s without sufficient test coverage will not be merged.
py.test as test runner but the tests themselves are Django based test classes. We have our own base class called
SocialhomeTestCase which should be used as a base for all Django tests. Alternatively, to test Django class based views, you can use our
SocialhomeCBVTestCase. Both these base classes inherit from the powerful django-test-plus test classes. Some old tests are pure py.test function based tests, feel free to convert these.
Focus is placed in pure unit tests instead of complex integration or browser tests. In terms of coverage, 100% is not the key, meaningful tests and coverage of critical lines is. Don’t worry if a PR drops coverage a bit if the coverage diff clearly shows all critical code paths are covered by meaningful tests.
As a general rule, for Python code follow PEP8, except with a 120 character line length. We provide an
.editorconfig in the repository root.
There is an
.eslintrc configuration provided as well. NOTE! This is only valid for the new Vue based frontend, not JS in