Conversation
|
Thanks for the review! I’ll complete the remaining changes you mentioned (docs and tests) by today. Apologies for the delay , I’ll update everything by tonight. |
|
Hey @Chandana098-learn, are able to make those changes? |
|
Hey! I’ve been working on the changes, but I’m currently facing issues with passing the flake8 and pytest checks. I’m still in my 1st year and learning, so I’d appreciate any guidance you can share. I’ll keep working on it and update you soon. Thanks! |
|
Those issues should be fixed by my previous review comments, so focus on those first |
|
hello, i have reviewed the changes. please look into it. |
for more information, see https://pre-commit.ci
|
Please check the changes you asked for |
Co-authored-by: cobalt <61329810+cobaltt7@users.noreply.github.com>
for more information, see https://pre-commit.ci
|
is there anything i need to do right now? |
|
Nope! It looks like some things were messed up with the markdown formatting, but I fixed them. Just waiting for CI to pass and I can merge. Thanks for the contribution! |
|
Thank you for the review and guidance! |
Closes #4820
Closes #4992
I updated the parse error message while still keeping it in readable format.
New Change: Now it shows the error in multiple lines , points to the error position with ^.