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

[4.x] examples removal #9034

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

senivam
Copy link
Member

@senivam senivam commented Jul 24, 2024

Description

(the description is ported with modifications from the 3.x examples removal) PR)

Example removal. Helidon 4 examples now live in the helidon-examples repository

This PR:

  1. Deletes examples from Helidon 4
  2. Updates the examples top-level README to direct users to helidon-examples
  3. Updates the examples profile to include a module called helidon-examples (instead of examples)
  4. Adds a build-examples script that clones the helidon-examples repository into the helidon-examples directory (if it doesn't already exist) and builds the examples.
  5. Adds helidon-examples to .gitignore

The last three items establish a pattern that can be used by developers to more easily work on changes to examples and core helidon concurrently.

Developing Examples

To make it a bit easier working in two different repositories (helidon and helidon-examples) you can use this workflow:

  1. Checkout your 4.x branch and build it
  2. Run etc/scripts/build-examples.sh. This will clone the examples repository's dev-4.x branch into helidon-examples and build it
  3. In your IDE under Maven configuration select the examples profile.

If you need to coordinate a (possibly breaking) change in the main helidon repo and in helidon-examples

  1. Create your branch in both local repositories
  2. Create pull requests against both repositories.
  3. The PR against helidon might fail the examples validation. But it will not block the PR.
  4. The PR against helidon-examples might fail validation. If so it will block the PR.
  5. Merge the PR against helidon
  6. Re-trigger validation on the helidon-examples PR which should now pass. Then merge.

@senivam senivam added examples 4.x Version 4.x labels Jul 24, 2024
@senivam senivam self-assigned this Jul 24, 2024
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jul 24, 2024
Signed-off-by: Maxim Nesen <[email protected]>
Signed-off-by: Maxim Nesen <[email protected]>
@senivam senivam requested a review from barchetta August 2, 2024 15:59
@romain-grecourt romain-grecourt merged commit 6405b02 into helidon-io:main Aug 2, 2024
13 checks passed
@senivam senivam deleted the 4x_examples_extraction branch August 2, 2024 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x examples OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants