DOC: adding docstring to TooHardError class#21471
Conversation
…error DOC: adding docstring to TooHardError class
|
This looks generally good to me. @melissawm Want to take a look. @angellicacardozo You are almost set up right, but not quite. We can work with this, but in the future you will want to make a new branch. If you currently have main checked out, make a new branch like so If you are working in your local branch, you then push to your own github repo Then make a pull request for that branch. You will also want to start with a clean repository next time. You can either redo the clone and all that bit, which is the easiest if you are not experienced, or you can try removing this commit and cleaning up from there. We can walk you through the process if you choose the latter. |
melissawm
left a comment
There was a problem hiding this comment.
Left a few optional comments, overall looks ok - @angellicacardozo let us know if you need help with setting up your branch :)
Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com>
Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com>
|
@melissawm @InessaPawson, suggestions were applied in the latest commit. I hope you appreciate it. Also, conversations are automatically resolved by GitHub when using suggested changes. Should I keep it open and wait for you to mark it as resolved? Please let me know what your thoughts are. Regards |
|
Looking to to me - thanks @angellicacardozo ! |
issue: 10106
Changes were made according to the previous request for a docstring to the class TooHardError.
Please let me know if any further changes are required,
Thanks for your attention