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

updated native methods guide #167

Merged
merged 8 commits into from
Nov 25, 2024
Merged

Conversation

theodoravraimakis
Copy link
Contributor

No description provided.

Copy link
Member

@epanholz epanholz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks very good overall, but a few things could be a bit clearer. Please update the file with my suggestions.

Copy link
Member

@epanholz epanholz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one looks good to me know. For the future please --amend to the first commit if I request some changes, to avoid unnecessary commits. Keeps the history clean.

Copy link
Contributor

@GabrielaReyna GabrielaReyna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR and great work. I have suggested a couple ofchanges regarding formatting.

I'd like to ask about the difference between this guide and "Implementing native methods and libraries". We will probably need to rename these guides to make it less confusing for the user since them two have very similar titles.

@epanholz
Copy link
Member

We decided to split the "Implementing native methods and libraries" guide into two separate ones, we heard from some customer that it is a bit difficult to understand so we also revamped them a bit. There is another PR with the standalone libraries guide.

theodoravraimakis and others added 6 commits November 21, 2024 14:27
…ethods.md

Co-authored-by: Ana Gabriela Reyna <gabrielareynaflores@gmail.com>
…ethods.md

Co-authored-by: Ana Gabriela Reyna <gabrielareynaflores@gmail.com>
…ethods.md

Co-authored-by: Ana Gabriela Reyna <gabrielareynaflores@gmail.com>
…ethods.md

Co-authored-by: Ana Gabriela Reyna <gabrielareynaflores@gmail.com>
…ethods.md

Co-authored-by: Ana Gabriela Reyna <gabrielareynaflores@gmail.com>
…ethods.md

Co-authored-by: Ana Gabriela Reyna <gabrielareynaflores@gmail.com>
@GabrielaReyna GabrielaReyna merged commit fad0090 into main Nov 25, 2024
9 checks passed
@GabrielaReyna GabrielaReyna deleted the native-methods-guide-update branch November 25, 2024 14:01
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.

3 participants