Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NF: explaining the bridge between languages #3572

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Arthur-Milchior
Copy link
Contributor

It was, honestly, quite confusing. And if I got it wrong, I guess this code review will be the time for me to learn what was wrong.

@dae
Copy link
Member

dae commented Nov 17, 2024

The technical details appear to be correct. This reads like some rough notes though, and needs some polishing before it's ready for inclusion (things like "as far as I understand" don't really belong in a document that is telling people what to do!)

It was, honestly, quite confusing. And if I got it wrong, I guess this
code review will be the time for me to learn what was wrong.
@Arthur-Milchior
Copy link
Contributor Author

Appart from the sentence you quoted, is there anything you believe needs more polishing? I would have hoped everything is understandable.
I'm happy you confirm everything seems to be correct, it took me some time to understand what I wrote down. Hope it'll help other devs. And great that my "as far as I understand" ensured you confirmed the understanding

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants