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

Fix invalid example in se/config/advanced-configuration.adoc (#4775) #4920

Merged

Conversation

Captain1653
Copy link
Contributor

@Captain1653 Captain1653 commented Sep 20, 2022

Part of #4775

I can backport it into 2.x and 4.x after merging if necessary.

I've already signed OCA. It's an automation problem.

@romain-grecourt
Copy link
Contributor

/oca-checked

@romain-grecourt
Copy link
Contributor

/trigger

@Captain1653
Copy link
Contributor Author

@romain-grecourt Can it be merged or we should wait for something?

@tjquinno
Copy link
Member

@Captain1653 The ports to 2.x and 4.x will need to be in separate PRs; a single PR can merge only into one branch.

I've added check boxes for each Helidon release to the issue so we can track the separate PRs there.

@Captain1653
Copy link
Contributor Author

Captain1653 commented Sep 23, 2022

@tjquinno Thank you for your explanation. I was waiting for merging this PR, my mistake. I'll create backports soon.

@romain-grecourt romain-grecourt merged commit 5b8d349 into helidon-io:helidon-3.x Sep 23, 2022
@Captain1653 Captain1653 deleted the 4775-config-builder branch September 23, 2022 18:45
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