EnvisEdge
EnvisEdge copied to clipboard
Added and improved inline documentation
Description
Added inline documentation for saver_utils.py as instructed in the issue.
Relevant Issue
Issue #193
Affected Dependencies
List any dependencies that are required for this change.
How has this been tested?
- Describe the tests that you ran to verify your changes.
- Provide instructions so we can reproduce.
- List any relevant details for your test configuration.
Checklist
- [x] I have followed the Contribution Guidelines and Code of Conduct
- [x] I have commented my code following the NimbleEdge Styleguide
- [x] I have labeled this PR with the relevant Type labels
- [x] My changes are covered by tests
Deploy Preview for goofy-agnesi-4565cb canceled.
Name | Link |
---|---|
Latest commit | 801dd2f8c5908fd77c072134868f019850373af3 |
Latest deploy log | https://app.netlify.com/sites/goofy-agnesi-4565cb/deploys/6273cdc4ad020f0008a66d0c |
@imanishbarnwal Can you please follow the Pep8 guidelines!!
Hey @divyanshugit, I fixed the Pep8 issue but still, the Pytests error is there.
Right now, You don't have to worry about the PyTest error. Ramesht is working on that.
Okay thanks, so can I put this PR in my proposal and form? @divyanshugit
It's your decision. If you think, it will be helpful then you can add it.
You can also take reference from the current docs !!
Hey @ramesht007, I am not able to fix this Pytests unittests
error rest are done. Kindly help me with this.
@imanishbarnwal You don't have to worry about it. He will take care of PyTest error
@imanishbarnwal You don't have to worry about it. He will take care of PyTest error
Okay sure, thanks.