Closed lapetina9 closed 4 years ago
What current issue(s) does this address, or what feature is it adding? Current documentation is not very user friendly
How did you solve this problem? I reviewed it with @lock9, changed some texts and fixed one broken link.
How did you make sure your solution works? I have little background in this area, I tried to make it more simple for starters (with @lock9 help).
Are there any special changes in the code that we should be aware of? No
Please check the following, if applicable:
make lint
make test
CHANGELOG.rst
How can I add it to changelog? Should I create a new version?
Coverage remained the same at 85.313% when pulling 3c1c727df7eda050d6c86c840c7867170de26063 on lapetina9:review-overview into 5fe7d51ef14614ff1cd0681f8fe2cb3e2cb76557 on CityOfZion:master.
Hi @ixje, I've made the changes you've requested. Let me know if you'd like to add or update anything else. Thanks!
Thank you for your contribution 👍
What current issue(s) does this address, or what feature is it adding? Current documentation is not very user friendly
How did you solve this problem? I reviewed it with @lock9, changed some texts and fixed one broken link.
How did you make sure your solution works? I have little background in this area, I tried to make it more simple for starters (with @lock9 help).
Are there any special changes in the code that we should be aware of? No
Please check the following, if applicable:
make lint
?make test
?CHANGELOG.rst
? (if not, please do)How can I add it to changelog? Should I create a new version?