| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
Hi Eric, I know we already spoke about some of this off-line, but I ran out of time reviewing all my comments. I know the next revision will be much improved!
Sorry, something went wrong.
|
@gvanrossum, I've made updates to address your review comments. Thanks again for taking the time to provide such useful feedback. ❤️ I'm concerned that the PEP is getting a little too long, so let me know if there are any sections that you think are unnecessary or have too much detail, if you can spare me the time. |
Sorry, something went wrong.
|
FYI, once this PR is merged, I plan on updating the implementation (and update some benchmarks, to try it out) before posting the PEP for discussion. That way I can tweak the PEP if something doesn't feel right in practice. |
Sorry, something went wrong.
|
Are you awaiting more review from me? Im in Japan but can make a little time. |
Sorry, something went wrong.
|
Sweet of you to offer, but no. If needed, we can chat when you are back. |
Sorry, something went wrong.
|
There are probably a handful of things to tweak, like trim back some of the fluff, but I'll do that in follow-up PRs. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Basic requirements (all PEP Types)
Standards Track requirements
📚 Documentation preview 📚: https://pep-previews--3523.org.readthedocs.build/pep-0734/