Skip to content
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

CAMEL-19998: cleanup unused methods #11770

Merged
merged 2 commits into from
Oct 19, 2023
Merged

CAMEL-19998: cleanup unused methods #11770

merged 2 commits into from
Oct 19, 2023

Conversation

orpiske
Copy link
Contributor

@orpiske orpiske commented Oct 19, 2023

No description provided.

@github-actions github-actions bot added the core label Oct 19, 2023
@github-actions
Copy link
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🤖 CI automation will test this PR automatically.

🐫 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 have access to the summary.

  • ⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@orpiske
Copy link
Contributor Author

orpiske commented Oct 19, 2023

See https://issues.apache.org/jira/browse/CAMEL-20014 for the removal ticket.

@essobedo
Copy link
Contributor

If you deprecate them to remove them later, maybe you should set the attribute forRemoval to true in your Deprecated annotations, WDYT?

@orpiske
Copy link
Contributor Author

orpiske commented Oct 19, 2023

If you deprecate them to remove them later, maybe you should set the attribute forRemoval to true in your Deprecated annotations, WDYT?

That's a good suggestion, thanks! Let's do that.

@orpiske orpiske merged commit a88f30c into apache:main Oct 19, 2023
3 checks passed
@orpiske orpiske deleted the camel-19998 branch April 10, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants