Closed mattdurant closed 1 month ago
LGTM, just a couple of minor things. @mattdurant wanted to ask if you're using
pre-commit
? Styling is automatically taken care of byruff
if youpip install pre-commit
and runpre-commit install
. (will add this to our future contributors guide :))
yes, and it has been doing reformatting for extraneous blank lines, etc.
@daryllimyt changes made
Description
Related Tickets & Documents
N/A
Screenshots/Recordings
N/A
Steps to QA
Tested against live API.
[optional] What gif best describes this PR?