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

chore(deps)!: update termion requirement from 3.0 to 4.0 #1106

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 13, 2024

Updates the requirements on termion to permit the latest version.

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

BREAKING CHANGE: Changelog: https://gitlab.redox-os.org/redox-os/termion/-/blob/master/CHANGELOG.md
A change is only necessary if you were matching on all variants of the MouseEvent enum without a
wildcard. In this case, you need to either handle the two new variants, MouseLeft and
MouseRight, or add a wildcard.

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label May 13, 2024
@dependabot dependabot bot requested a review from orhun as a code owner May 13, 2024 15:34
@dependabot dependabot bot added the rust Pull requests that update Rust code label May 13, 2024
Copy link
Sponsor Member

@orhun orhun left a comment

Choose a reason for hiding this comment

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

Looks good to merge: https://gitlab.redox-os.org/redox-os/termion/-/blob/master/CHANGELOG.md

Not sure if it is too breaking though.

@EdJoPaTo
Copy link
Member

Not sure if it is too breaking though.

Guess that should wait till after the span bug fix release.

@joshka joshka added this to the v0.27.0 milestone May 13, 2024
@dependabot dependabot bot force-pushed the dependabot/cargo/termion-4.0 branch from 496792f to fcec1bc Compare May 14, 2024 01:14
@dependabot dependabot bot force-pushed the dependabot/cargo/termion-4.0 branch from fcec1bc to 7d72ee6 Compare May 25, 2024 17:39
@joshka
Copy link
Member

joshka commented Jun 1, 2024

Not sure if it is too breaking though.

From the Changelog:

A change is only necessary if you were matching on all variants of the MouseEvent enum without a wildcard.
In this case, you need to either handle the two new variants, MouseLeft and MouseRight, or add a wildcard.

So this is breaking and would potentially break ratatui users, and should be noted in the breaking changes doc and commit

@joshka
Copy link
Member

joshka commented Jun 3, 2024

@dependabot rebase

@dependabot dependabot bot force-pushed the dependabot/cargo/termion-4.0 branch from 7d72ee6 to 0ebf5c3 Compare June 3, 2024 01:19
@joshka
Copy link
Member

joshka commented Jun 3, 2024

@dependabot rebase

@joshka joshka force-pushed the dependabot/cargo/termion-4.0 branch 2 times, most recently from cf548a7 to 323b8de Compare June 3, 2024 01:22
@joshka joshka force-pushed the dependabot/cargo/termion-4.0 branch from 30c8f4a to f2b50bb Compare June 3, 2024 01:27
@joshka joshka merged commit 04032dd into main Jun 3, 2024
34 of 36 checks passed
@joshka joshka deleted the dependabot/cargo/termion-4.0 branch June 3, 2024 01:30
@joshka joshka changed the title chore(deps): update termion requirement from 3.0 to 4.0 chore(deps)!: update termion requirement from 3.0 to 4.0 Jun 3, 2024
joshka added a commit that referenced this pull request Jun 3, 2024
Updates the requirements on termion to permit the latest version.

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)

</details>

BREAKING CHANGE: Changelog:
<https://gitlab.redox-os.org/redox-os/termion/-/blob/master/CHANGELOG.md>
A change is only necessary if you were matching on all variants of the
`MouseEvent` enum without a
wildcard. In this case, you need to either handle the two new variants,
`MouseLeft` and
`MouseRight`, or add a wildcard.

Co-authored-by: Josh McKinney <joshka@users.noreply.github.com>
@joshka
Copy link
Member

joshka commented Jun 3, 2024

I forced pushed a change that added the ! to the title to make sure it shows up in the changelog correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change dependencies Pull requests that update a dependency file do not merge rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants