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

Feature/m0081 meja 2367 managedc friviere review #1155

Conversation

frivieremicroej
Copy link
Contributor

No description provided.

Copy link
Collaborator

@dalgassimou dalgassimou left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

Choose a reason for hiding this comment

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

"C Compiler (WASI SDK)" ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove any reference to a specific compiler in the diagram (JDT, etc.)

ApplicationDeveloperGuide/runtime.rst Outdated Show resolved Hide resolved
frivieremicroej and others added 9 commits December 24, 2024 09:58
- languages are not exclusives per app
- Managed/Unmanaged code are not boxes of the VEE diagram and conflict with the text on the right
this is not a library like JavaScript
add build flow
also merge Java limitations in java language subchapter
binary code verifier and also introduce SOAR build flow
@jerome-leroux jerome-leroux force-pushed the feature/M0081MEJA-2367_managedc-friviere-review branch from a6993f4 to 5949689 Compare December 24, 2024 08:59
@jerome-leroux jerome-leroux merged commit 85bad87 into feature/M0081MEJA-2367_managedc Dec 24, 2024
1 check passed
@jerome-leroux jerome-leroux deleted the feature/M0081MEJA-2367_managedc-friviere-review branch December 24, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants