Skip to content

remove method#199

Merged
cornelcroi merged 3 commits into
mainfrom
remove-set-classifier
Jan 10, 2025
Merged

remove method#199
cornelcroi merged 3 commits into
mainfrom
remove-set-classifier

Conversation

@cornelcroi
Copy link
Copy Markdown
Collaborator

Issue number:

Summary

Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented
Is this a breaking change?

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@cornelcroi cornelcroi requested a review from brnaba-aws January 9, 2025 19:43
Copy link
Copy Markdown
Collaborator

@brnaba-aws brnaba-aws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The doc needs to be updated as well

@cornelcroi
Copy link
Copy Markdown
Collaborator Author

Doc updated

@cornelcroi cornelcroi merged commit dd9f5a2 into main Jan 10, 2025
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.

2 participants