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

docs: remove simple-tui-rs #651

Merged
merged 1 commit into from
Dec 2, 2023
Merged

docs: remove simple-tui-rs #651

merged 1 commit into from
Dec 2, 2023

Conversation

joshka
Copy link
Member

@joshka joshka commented Nov 27, 2023

This has not been recently and doesn't lead to good code

@orhun
Copy link
Sponsor Member

orhun commented Nov 27, 2023

Maybe we can ping the maintainer of that repo here and let them know before merging this? (in case they might want to update the template)

This has not been recently and doesn't lead to good code
@orhun
Copy link
Sponsor Member

orhun commented Nov 27, 2023

ping @pmsanford

@joshka
Copy link
Member Author

joshka commented Nov 27, 2023

The context on this is that a user in discord mentioned they were using this repo as a base because it seemed like the right idea. It may have been way back when tui-rs was just a young project, but it's got a few things would be better not to use in a newer project.

  • Termion
  • Spans instead of Line
  • etc.

@pmsanford
Copy link

Thanks for the ping; I'd be happy to update it, but I can't make any commitments on a timeline at the moment. I'd suggest going ahead with merging this PR and when/if I get it updated I'll open one to re-add it. I'll also add a note to my repo saying it's not up to date and link to this PR.

@orhun
Copy link
Sponsor Member

orhun commented Nov 27, 2023

Sounds good, thank you!

@orhun orhun merged commit 211160c into main Dec 2, 2023
33 checks passed
@orhun orhun deleted the remove-simple-tui-rs branch December 2, 2023 15:17
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.

None yet

3 participants