You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "squakez (via GitHub)" <gi...@apache.org> on 2024/02/27 08:26:24 UTC

[PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

squakez opened a new pull request, #5187:
URL: https://github.com/apache/camel-k/pull/5187

   <!-- Description -->
   
   It is flaky when the number of dependencies matching is odd. Adding unit test to prove it in different situations.
   
   
   
   <!--
   Enter your extended release note in the below block. If the PR requires
   additional action from users switching to the new release, include the string
   "action required". If no release note is required, write "NONE". 
   
   You can (optionally) mark this PR with labels "kind/bug" or "kind/feature" to make sure
   the text is added to the right section of the release notes. 
   -->
   
   **Release Note**
   ```release-note
   fix(builder): strenghten matching dependencies heuristic
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #5187:
URL: https://github.com/apache/camel-k/pull/5187#issuecomment-1966031696

   :heavy_check_mark: Unit test coverage report - coverage increased from 36% to 36.2% (**+0.2%**)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #5187:
URL: https://github.com/apache/camel-k/pull/5187#issuecomment-1966067399

   > @squakez can you add some better release notes ? it is not clear to me what is the motivation of this change and how it differs form the current behavior and what would be the impact.
   
   Sure. Done. Thanks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #5187:
URL: https://github.com/apache/camel-k/pull/5187#issuecomment-1968721430

   :heavy_check_mark: Unit test coverage report - coverage increased from 36% to 36.2% (**+0.2%**)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

Posted by "lburgazzoli (via GitHub)" <gi...@apache.org>.
lburgazzoli commented on PR #5187:
URL: https://github.com/apache/camel-k/pull/5187#issuecomment-1966050426

   @squakez can you add some better release notes ? it is not clear to me what is the motivation of this change and how it differs form the current behavior


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #5187:
URL: https://github.com/apache/camel-k/pull/5187#issuecomment-1966020654

   Part of #5090 - FYI @lburgazzoli 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(builder): strenghten matching dependencies heuristic [camel-k]

Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez merged PR #5187:
URL: https://github.com/apache/camel-k/pull/5187


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org