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.

Sign up for GitHub

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

Jump to bottom

chore!: 🔥 Remove deprecated support for Option in bridge commands #2731

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

chore!: 🔥 Remove deprecated support for Option in bridge commands #2731

wants to merge 3 commits into from

Conversation

Paillat-dev
Copy link
Contributor

@Paillat-dev Paillat-dev commented Mar 5, 2025 •
edited
Loading

Summary

Fixes #2451
If still planned, can be merged before 2.7 is released. If it is preferred not to raise an error at all, let me know.

Information

  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed).
  • This PR is not a code change (e.g. documentation, README, typehinting,
    examples, ...).

Checklist

  • I have searched the open pull requests for duplicates.
  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why.
  • I have updated the changelog to include these changes.

@Paillat-dev Paillat-dev requested review from a team as code owners March 5, 2025 19:51
@pullapprove4 pullapprove4 bot requested a review from Dorukyum March 5, 2025 19:51
@Paillat-dev Paillat-dev changed the title chore: 🔥 Remove deprecated support for Option in bridge commands chore!: 🔥 Remove deprecated support for Option in bridge commands Mar 5, 2025
@Paillat-dev Paillat-dev marked this pull request as draft March 5, 2025 19:58
@Paillat-dev Paillat-dev marked this pull request as ready for review March 9, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dorukyum Dorukyum Awaiting requested review from Dorukyum

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
PA: All Contributors pending PA: Maintainers pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

v2.7: Remove backwards support for Option in bridge commands.
1 participant
@Paillat-dev