Thank you very much. I appreciate the pull request, but I will need to close it because of these reasons:
Tests don't pass.
It's a huge pull request, which should have been split into 7-8 separate pull requests. Due to this, it takes a lot of work to review.
I saw some things that shouldn't have been done. For example, the .gitignore file should have stayed the same.
Please submit each pull request for a single change. For example, if you modify the comments in a Python file, send a pull request exclusively for that update. This will simplify my review and help me determine whether to merge.
Pull requests should be like this:
Update Issue Template
Update .gitignore
Update docstrings of renderer.py
Update comments of renderer.py
etc.
Maybe you can update this pull request accordingly or open a new one.
Thank you very much. I appreciate the pull request, but I will need to close it because of these reasons:
.gitignore
file should have stayed the same.Please submit each pull request for a single change. For example, if you modify the comments in a Python file, send a pull request exclusively for that update. This will simplify my review and help me determine whether to merge.
Pull requests should be like this:
.gitignore
renderer.py
renderer.py
Maybe you can update this pull request accordingly or open a new one.