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 2023/05/03 14:46:28 UTC
[GitHub] [camel-k] squakez opened a new pull request, #4318: feat: remove deprecated code
squakez opened a new pull request, #4318:
URL: https://github.com/apache/camel-k/pull/4318
Close #4026
<!-- Description -->
<!--
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
feat: remove deprecated code
```
--
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
[GitHub] [camel-k] squakez commented on pull request #4318: feat: remove deprecated code
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #4318:
URL: https://github.com/apache/camel-k/pull/4318#issuecomment-1533208474
> Should the tracing trait also be removed now ?
Not so easy. As it is part of the `v1` API spec that we still support, we cannot remove it. API removal should be only be done in a major API upgrade. In Camel K V2, we're not upgrading Kubernetes API, so, we better leave it (although end users should not use the feature) for backward compatibility only.
--
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
[GitHub] [camel-k] squakez commented on pull request #4318: feat: remove deprecated code
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez commented on PR #4318:
URL: https://github.com/apache/camel-k/pull/4318#issuecomment-1541875541
Check failure due to regression to be fixed by #4358. Merging.
--
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
[GitHub] [camel-k] squakez merged pull request #4318: feat: remove deprecated code
Posted by "squakez (via GitHub)" <gi...@apache.org>.
squakez merged PR #4318:
URL: https://github.com/apache/camel-k/pull/4318
--
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
[GitHub] [camel-k] gansheer commented on pull request #4318: feat: remove deprecated code
Posted by "gansheer (via GitHub)" <gi...@apache.org>.
gansheer commented on PR #4318:
URL: https://github.com/apache/camel-k/pull/4318#issuecomment-1533189888
Should the tracing trait also be removed now ?
--
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