Skip to content

feat(simulator)!: remove simulator feature#83

Merged
j-g00da merged 1 commit intoratatui:mainfrom
deadbaed:simulator_example
Jun 22, 2025
Merged

feat(simulator)!: remove simulator feature#83
j-g00da merged 1 commit intoratatui:mainfrom
deadbaed:simulator_example

Conversation

@deadbaed
Copy link
Copy Markdown
Contributor

@deadbaed deadbaed commented Jun 19, 2025

This completely supersedes #67.

  • moved simulator from mousefood/examples to examples/simulator
  • removed simulator feature
  • ci: run fmt and clippy on all workspace crates

@j-g00da
Copy link
Copy Markdown
Member

j-g00da commented Jun 19, 2025

Rebase the branch, markdownlint ci didn't work, it's fixed now.

@j-g00da j-g00da changed the title feat(simulator)!: removed simulator feature feat(simulator)!: remove simulator feature Jun 19, 2025
@deadbaed deadbaed force-pushed the simulator_example branch from 07e6918 to 2ed04eb Compare June 19, 2025 16:42
@deadbaed
Copy link
Copy Markdown
Contributor Author

A 80 columns limit in a readme file seems a bit extreme for me, but i'll fix that.

@deadbaed deadbaed force-pushed the simulator_example branch from 2ed04eb to 9222503 Compare June 19, 2025 16:48
@deadbaed deadbaed requested a review from j-g00da June 19, 2025 16:50
@j-g00da
Copy link
Copy Markdown
Member

j-g00da commented Jun 19, 2025

Yeah, that's the default setting, I would change it to 100

Copy link
Copy Markdown
Member

@j-g00da j-g00da left a comment

Choose a reason for hiding this comment

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

Left some nitpick comments, but generally LGTM.

@j-g00da j-g00da linked an issue Jun 19, 2025 that may be closed by this pull request
@deadbaed deadbaed force-pushed the simulator_example branch from 9222503 to ac3b8bb Compare June 19, 2025 18:26
@deadbaed
Copy link
Copy Markdown
Contributor Author

Whoops, I'm too used of having editorconfig with the trim_trailing_whitespace rule, my bad

@deadbaed deadbaed force-pushed the simulator_example branch from ac3b8bb to 64bec4b Compare June 19, 2025 18:31
- moved simulator from mousefood/examples to examples/simulator
- removed `simulator` feature
- ci: run `fmt` and `clippy` on all workspace crates
@deadbaed deadbaed force-pushed the simulator_example branch from 64bec4b to 1d2610f Compare June 19, 2025 18:35
@deadbaed deadbaed requested a review from j-g00da June 19, 2025 18:38
Copy link
Copy Markdown
Member

@j-g00da j-g00da left a comment

Choose a reason for hiding this comment

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

LGTM!

@j-g00da j-g00da merged commit 88c83fe into ratatui:main Jun 22, 2025
24 checks passed
@github-actions github-actions bot mentioned this pull request Jan 23, 2026
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.

Simulator feature flag is semver-incompatible

2 participants