Conversation
Edited for passing travis test
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: 36d37fe0-007b-11eb-a257-6fde496095bb |
now i am done |
|
please review my pull request |
Compressed the line 19 to 28 into 19 to 21
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: d1eeb870-014b-11eb-9be6-0d9634a5fbe7 |
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: 9c7a9f40-0152-11eb-9be6-0d9634a5fbe7 |
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: e07a4e70-0152-11eb-9be6-0d9634a5fbe7 |
completly corrected pep8 errors using Pycharm IDE
|
Edited code to get approved |
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: f6b7dc10-0153-11eb-9be6-0d9634a5fbe7 |
|
editing |
|
please review it |
Co-authored-by: Dhruv <dhruvmanila@gmail.com>
Co-authored-by: Dhruv <dhruvmanila@gmail.com>
Co-authored-by: Dhruv <dhruvmanila@gmail.com>
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: 42fdb2f0-0156-11eb-9be6-0d9634a5fbe7 |
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: 9d27f970-0156-11eb-9be6-0d9634a5fbe7 |
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: ce7ce940-0156-11eb-9be6-0d9634a5fbe7 |
|
done and approved by travis |
cclauss
left a comment
There was a problem hiding this comment.
Thanks for your perseverance we will get this one merged soon.
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
Co-authored-by: Christian Clauss <cclauss@me.com>
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: 66f0a560-015f-11eb-9be6-0d9634a5fbe7 |
Added A few extra names to __author__ 😄
Used Pycharm to fix PEP8 errors, doctest errors
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: 07eef0c0-0160-11eb-9be6-0d9634a5fbe7 |
Added enough doctests
|
It is very hard work to comply with all the requirements 😫. But still i am happy for contributions 😄 |
|
please review |
Co-authored-by: Christian Clauss <cclauss@me.com>
|
Hey @xcodz-dot, TravisCI finished with status TravisBuddy Request Identifier: e08d1450-016c-11eb-9be6-0d9634a5fbe7 |
cclauss
left a comment
There was a problem hiding this comment.
Nice work! Thanks for doing this.
* Create vector3_for_2d_rendering.py Edited for passing travis test * Delete vector3_for_2d_rendering.py * Create vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py Compressed the line 19 to 28 into 19 to 21 * Update vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py completly corrected pep8 errors using Pycharm IDE * Update vector3_for_2d_rendering.py * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Dhruv <dhruvmanila@gmail.com> * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Dhruv <dhruvmanila@gmail.com> * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Dhruv <dhruvmanila@gmail.com> * Update vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Christian Clauss <cclauss@me.com> * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Christian Clauss <cclauss@me.com> * Apply suggestions from code review Co-authored-by: Christian Clauss <cclauss@me.com> * Update vector3_for_2d_rendering.py Added A few extra names to __author__ 😄 * Update vector3_for_2d_rendering.py Used Pycharm to fix PEP8 errors, doctest errors * Update vector3_for_2d_rendering.py Added enough doctests * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Christian Clauss <cclauss@me.com> * Remove second main() Co-authored-by: Dhruv <dhruvmanila@gmail.com> Co-authored-by: Christian Clauss <cclauss@me.com>
* Create vector3_for_2d_rendering.py Edited for passing travis test * Delete vector3_for_2d_rendering.py * Create vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py Compressed the line 19 to 28 into 19 to 21 * Update vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py completly corrected pep8 errors using Pycharm IDE * Update vector3_for_2d_rendering.py * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Dhruv <dhruvmanila@gmail.com> * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Dhruv <dhruvmanila@gmail.com> * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Dhruv <dhruvmanila@gmail.com> * Update vector3_for_2d_rendering.py * Update vector3_for_2d_rendering.py * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Christian Clauss <cclauss@me.com> * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Christian Clauss <cclauss@me.com> * Apply suggestions from code review Co-authored-by: Christian Clauss <cclauss@me.com> * Update vector3_for_2d_rendering.py Added A few extra names to __author__ 😄 * Update vector3_for_2d_rendering.py Used Pycharm to fix PEP8 errors, doctest errors * Update vector3_for_2d_rendering.py Added enough doctests * Update graphics/vector3_for_2d_rendering.py Co-authored-by: Christian Clauss <cclauss@me.com> * Remove second main() Co-authored-by: Dhruv <dhruvmanila@gmail.com> Co-authored-by: Christian Clauss <cclauss@me.com>
Edited for passing travis test
Describe your change:
Checklist:
Fixes: #{$ISSUE_NO}.