site
site copied to clipboard
Subclassing context.md
Deploy Preview for pydis-static ready!
Name | Link |
---|---|
Latest commit | b6ec3f08c953ba7e4f865b6bdee9652d84aabbe0 |
Latest deploy log | https://app.netlify.com/sites/pydis-static/deploys/641a18cf6787400008b5647f |
Deploy Preview | https://deploy-preview-703--pydis-static.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site settings.
Coverage: 100.0%. Remained the same when pulling 0cf7fc78f075bb46ca2d9b93922e5ffe522296a7 on Diabolical5777:subclassing_context.md into 58ef0e327f7a956a4d7220fa9e652bf01488befa on python-discord:main.
Looks good to me! Though before this gets merged, you could implement some of Volccy's feedback.
Which ones? I thought I implemented them all
Looks good to me! Though before this gets merged, you could implement some of Volccy's feedback.
Which ones? I thought I implemented them all
I assumed since some of them weren't marked as resolved yet that they weren't completed, a mistake on my end haha
As per this message in the discord I will be taking over this PR.
Hey @Bluenix2, what's your status on this PR?
Thanks!
I do not have any other changes - this is ready for review! The preview failed because of ratelimits, I am not sure whether I can re-run them.
Very cool. Thanks!
Thanks for these reviews! I am hoping to get to this soon, sorry
Hey @Bluenix2 ! Don't mean to rush you, but what's your status on getting the reviews resolved? If you're unable to, we can make this up for grabs for someone else, no worries! Thanks 😄
wanted to rename the file for consistency, will need someone elses approval