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

fix: make all tests and crates compileable with ratatui 0.28 #24

Merged
merged 3 commits into from
Aug 9, 2024

Conversation

p0kR
Copy link
Contributor

@p0kR p0kR commented Aug 8, 2024

Working on one of my projects, I found out, I was not able to build with ratatui 0.28 and any version of tui-big-text.
With this pull request, I want to mak the crates from this repository compilable with ratatui 0.28.

With these two changes, I am able to build my project against ratatui 0.28 and tui-big-text from this branch. And all tests and crates from this repository can be built and run again.

After this branch is merged, all components need new versions, I think.

@codecov-commenter
Copy link

codecov-commenter commented Aug 8, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (a439170) to head (2e315be).

Files Patch % Lines
tui-scrollview/src/scroll_view.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #24   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          2       2           
  Lines        136     136           
=====================================
  Misses       136     136           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- Frame::size() has been replaced with Frame::area()
- Buffer index supports `Into<Position>`
- Put back the `default-features = false` for ratatui in Cargo.toml
  (this makes it possible to use the widgets on other backends than
  except tui-prompts which requires crossterm)
@joshka
Copy link
Owner

joshka commented Aug 9, 2024

Thanks. I've been meaning to add a dependabot setting that specifically pulls in Ratatui daily rather than weekly, but haven't gotten around to it yet. I updated a few things (deprecations, lints, and some more concise code), and this will be released shortly.

@joshka joshka merged commit def2598 into joshka:main Aug 9, 2024
11 checks passed
joshka pushed a commit that referenced this pull request Aug 9, 2024
## 🤖 New release
* `tui-big-text`: 0.5.4 -> 0.5.5

<details><summary><i><b>Changelog</b></i></summary><p>

## `tui-big-text`
<blockquote>

## [0.5.5] - 2024-08-09

### 🐛 Bug Fixes

- Update to ratatui 0.28
([#24](#24))
> Note that for projects that rely on crossterm, Ratatui 0.28.0 now
relies internally on Crossterm 0.28.0.
> Ratatui release notes highlights: https://ratatui.rs/highlights/v028/
> See ratatui/ratatui#1298 for notes about
crossterm compatibility
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
joshka pushed a commit that referenced this pull request Aug 9, 2024
## 🤖 New release
* `tui-widgets`: 0.2.4 -> 0.2.5

<details><summary><i><b>Changelog</b></i></summary><p>

## `tui-widgets`
<blockquote>

## [0.2.5] - 2024-08-09

### 🐛 Bug Fixes

- Add missing cfg ([#28](#28))

### ⚙️ Miscellaneous Tasks

- *(tui-big-text)* Release v0.5.5
([#25](#25))
  > ## 🤖 New release
  > * `tui-big-text`: 0.5.4 -> 0.5.5
  >
  > <details><summary><i><b>Changelog</b></i></summary><p>
  >
  > ## `tui-big-text`
  > <blockquote>
  >
  > ## [0.5.5] - 2024-08-09
  >
  > ### 🐛 Bug Fixes
  >
  > - Update to ratatui 0.28
>
([[#24](https://github.com/joshka/tui-widgets/pull/24)](https://github.com/joshka/tui-widgets/pull/24))
  > > Note that for projects that rely on crossterm, Ratatui 0.28.0 now
  > relies internally on Crossterm 0.28.0.
> > Ratatui release notes highlights:
https://ratatui.rs/highlights/v028/
> > See ratatui/ratatui#1298 for notes
about
  > crossterm compatibility
  > </blockquote>
  >
  >
  > </p></details>
  >
  > ---
  > This PR was generated with
  > [release-plz](https://github.com/MarcoIeni/release-plz/).
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants