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

[PR] CAMEL-21067: marked camel-univocity as deprecated [camel]

orpiske opened a new pull request, #15083:
URL: https://github.com/apache/camel/pull/15083

   (no comment)


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   Shouldn't it be added to the migration guide?


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   Changes done. Let's merge.


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   No its fine just to mark the component as deprecated as Otavio do. No need to do more.


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   Ah okay its a data format, so its a good idea to add `@Deprecated` to its model definition classes, so its also marked as deprecated in the DSL and in catalog and so on. So there is a bit more work when its a dataformat / language vs a regular component.


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   Let's mark this one as a draft. Later I'll take a look at the public classes we should deprecate. 


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   :star2: Thank you for your contribution to the Apache Camel project! :star2: 
   
   :robot: CI automation will test this PR automatically.
   
   :camel: Apache Camel Committers, please review the following items:
   
   * First-time contributors **require MANUAL approval** for the GitHub Actions to run
   
   * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot.
   
   * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR.
   
   * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. 
   
   * :warning: Be careful when sharing logs. Review their contents before sharing them publicly.


-- 
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] CAMEL-21067: marked camel-univocity as deprecated [camel]

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

   Also shouldn't we mark all public classes as Deprecated inside this module?


-- 
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