Skip to content

Conversation

@squakez
Copy link
Contributor

@squakez squakez commented Dec 14, 2025

We enable the possibility to issue a PR on a given repository with the Integration recently built, and allow the deployment with a GitOps approach

Closes #6137

We enable the possibility to issue a PR on a given repository with the Integration recently built, and allow the deployment with a GitOps approach

Closes apache#6137
@squakez
Copy link
Contributor Author

squakez commented Dec 14, 2025

@michalvavrik you may want to have a look at this as you were previously working on something similar. Feel free to provide any feedback.

@github-actions
Copy link
Contributor

⚠️ Unit test coverage report - coverage decreased from 51.6% to 50.6% (-1%)

@michalvavrik
Copy link
Contributor

@squakez thanks, I'll look at it to learn later this week, because I am interested how this could had been done. I won't provide any feedback since I don't have time next 2 days.

@squakez
Copy link
Contributor Author

squakez commented Dec 15, 2025

@squakez thanks, I'll look at it to learn later this week, because I am interested how this could had been done. I won't provide any feedback since I don't have time next 2 days.

No problem. I pinged you as you were working on it and likely you may be interested in how in the future Camel K will provide this feature. As you will see, this is completely operator based, so, the user won't need to do anything at all. The git part is still the one you have contributed on the CLI side.

@squakez squakez marked this pull request as draft December 15, 2025 08:13
@squakez
Copy link
Contributor Author

squakez commented Dec 15, 2025

Converting to draft as the PR lowered the coverage by 1%. Need to understand where and have a more complete coverage.

@squakez squakez marked this pull request as ready for review December 15, 2025 17:22
@squakez
Copy link
Contributor Author

squakez commented Dec 15, 2025

The lowered coverage is not real. The calculation of the percentage is wrong. I'm opening a follow up issue to fix that.

@squakez squakez merged commit 1605d15 into apache:main Dec 15, 2025
11 of 12 checks passed
@squakez squakez deleted the feat/6137 branch December 15, 2025 17:24
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.

Integration git, create a PR with integration yaml result after build

2 participants