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

[GR-60094] Native Image Public Demos Status Review and Refactoring. Part 1 #300

Merged
merged 10 commits into from
Nov 28, 2024

Conversation

olyagpl
Copy link
Member

@olyagpl olyagpl commented Nov 26, 2024

The goal is to sort the demos by categories (Native Image, referenced from blog posts, archived, microservices, etc.)
In this PR we:

  1. Create a subfolder /native-image/ and move some Native Image demos there (these demos correspond to the guides at https://www.graalvm.org/latest/guides/)
  2. Add new demos that were missing (some Native Image guides examples were missing or these guides have been just recently added)
  3. Clean demos README, they should point to the corresponding guide at https://www.graalvm.org/latest/guides/ instead of maintaining the two versions
  4. Add run.sh scripts for each demo for users convenience and for general consistency
  5. Sort the workflow files by placing those demos files' into a subfolder .github/workflows/native-image

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Nov 26, 2024
@olyagpl olyagpl force-pushed the ni-demos-refactoring branch 4 times, most recently from fd5c5db to cf742b1 Compare November 27, 2024 18:59
@olyagpl olyagpl force-pushed the ni-demos-refactoring branch from cf742b1 to b929c68 Compare November 27, 2024 19:00
@olyagpl olyagpl changed the title [GR-60094] Native Image Public Demos Status Review and Refactoring [WIP] [GR-60094] Native Image Public Demos Status Review and Refactoring. Part 1 Nov 27, 2024
@olyagpl olyagpl requested a review from shaunsmith November 28, 2024 16:38
Copy link
Contributor

@shaunsmith shaunsmith left a comment

Choose a reason for hiding this comment

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

Looks good, based on our Zoom review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants