-
Notifications
You must be signed in to change notification settings - Fork 350
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
(#5489) Groovy > Yaml #5497
(#5489) Groovy > Yaml #5497
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. However we must not add any file example here directly. For those we have the https://github.com/apache/camel-k-examples/tree/main/generic-examples
All along the documentation we just need to replace the snippets with the yaml DSL, that should be enough. And yes, feel free to open separate PR and do it iteratively if you prefer.
Do you know what this is ...
|
No, unfortunately not. We have some configuration magic that I am not familiar with. Mind that certain configuration are required by the main Camel project which is in charge to include the subproject (hence, Camel K among others) bits. |
docs/modules/ROOT/pages/index.adoc
Outdated
|
||
```yaml | ||
- from: | ||
uri: "timer:yaml" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd leave the timer:tick
instead. Any newcomer may confuse the name of the component with the DSL.
75d792c
to
b36622a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor changes, the rest is good, thanks!
docs/modules/ROOT/pages/configuration/build-time-properties.adoc
Outdated
Show resolved
Hide resolved
docs/modules/ROOT/pages/configuration/build-time-properties.adoc
Outdated
Show resolved
Hide resolved
docs/modules/ROOT/pages/configuration/build-time-properties.adoc
Outdated
Show resolved
Hide resolved
|
28321b3
to
2961aa0
Compare
All docs should be done - ready to merge |
If we merge, let's run the ci-build for website here: https://ci-builds.apache.org/job/Camel/job/Camel.website/job/main/ in case something is broken we need to fix it soon or revert. |
|
This is good, but the full build from the website on the CI might fail anyway. It happened before. But this is great! Thanks @tdiesler |
Could we run this again pls? |
Ready to merge |
Thanks @oscerd - the msg for the squashed commit is a little awkward |
Sorry for that. |
The build seems to be fine. The website should be online in a few minutes |
@tdiesler we did changes in autogenerated code in traits. It is likely that tonight it will be reverted back by auto updates. You need to change the content of the trait golang files and later |
I rebased those changes on main and see no differences. I believe I already changed stuff at the source i.e. the trait Go files. Also, I'm having trouble running |
See:
We need to make the change on the go file and then, |
Like so ... |
There will be so many changes ... so perhaps we do this incrementally