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

Add Java sections to best practices #40

Merged
merged 2 commits into from
Mar 6, 2025

Conversation

thomasleplus
Copy link
Contributor

This PR is a proposal documentation for Java best practices regarding the usage of non memory-safe Java APIs and interfacing with non-memory safe languages.

@thomasleplus
Copy link
Contributor Author

Contributes to #36.

@nellshamrell
Copy link
Contributor

Most excellent! ty! we will review at the meeting tomorrow :)

@GabrielDosReis
Copy link

Looks good to me.

@balteravishay balteravishay merged commit 7dc4e84 into ossf:main Mar 6, 2025
2 checks passed
balteravishay pushed a commit to balteravishay/Memory-Safety that referenced this pull request Mar 20, 2025
Signed-off-by: Thomas Leplus <[email protected]>
Signed-off-by: balteravishay <[email protected]>
balteravishay added a commit that referenced this pull request Mar 21, 2025
* draft

Signed-off-by: balteravishay <[email protected]>

* lint

Signed-off-by: balteravishay <[email protected]>

* pr comments

Signed-off-by: balteravishay <[email protected]>

* Update docs/memory-safety-continuum/blog-draft.md

Signed-off-by: Avishay Balter <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* Update link placeholder in blog draft

Signed-off-by: balteravishay <[email protected]>

* add some rust best practices for interop

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* adds general best practice

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* fix markdown

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* more markdown fixes

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* Update docs/best-practice-interfacing.md

Signed-off-by: Nell Shamrell-Harrington <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* Update docs/best-practice-interfacing.md

Co-authored-by: Avishay Balter <[email protected]>
Signed-off-by: Nell Shamrell-Harrington <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* Add Java sections to best practices (#40)

Signed-off-by: Thomas Leplus <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* add additional Rust practice (#41)

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* adds more language about interfacing between memory safe by default and non-memory safe by default languages

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* fix markdown

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* removes specific interop practices, links to best practices document

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* Remove language-specific best practices section

Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: balteravishay <[email protected]>

* release document

Signed-off-by: balteravishay <[email protected]>

* fixes

Signed-off-by: balteravishay <[email protected]>

* blog links

Signed-off-by: balteravishay <[email protected]>

---------

Signed-off-by: balteravishay <[email protected]>
Signed-off-by: Avishay Balter <[email protected]>
Signed-off-by: Nell Shamrell <[email protected]>
Signed-off-by: Nell Shamrell-Harrington <[email protected]>
Signed-off-by: Thomas Leplus <[email protected]>
Co-authored-by: Nell Shamrell <[email protected]>
Co-authored-by: Thomas Leplus <[email protected]>
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.

4 participants