ratatui/CONTRIBUTING.md
Leon Sautour bc5a9e4c06
doc: Updated readme and contributing guide with updates about the fork (#46)
* doc: Updated readme and contributing guide with updates about the fork

* doc: added missing discord invite link
2023-02-13 16:10:08 +01:00

2.8 KiB

Fork Status

Pull Requests

All pull requests opened on the original repository have been imported. We'll be going through any open PRs in a timely manner, starting with the smallest bug fixes and REAMDE updates. If you have an open PR make sure to let us know about it on our discord as it helps to know you are still active.

Issues

We have been unsuccessfull in importing all issues opened on the previous repository. For that reason, anyone wanting to work on or discuss an issue will have to follow the following workflow :

  • Recreate the issue
  • Start by referencing the original issue: Referencing issue #[<issue number>](<original issue link>)
  • Then, paste the original issues opening text

You can then resume the conversation by replying to this new issue you have created.

Closing Issues

If you close an issue that you have "imported" to this fork, please make sure that you add the issue to the CLOSED_ISSUES.md. This will enable us to keep track of which issues have been closed from the original repo, in case we are able to have the original repository transfered.

Contributing

Building

tui is an ordinary Rust project where common tasks are managed with cargo-make. It wraps common cargo commands with sane defaults depending on your platform of choice. Building the project should be as easy as running cargo make build.

🛠️ Pull requests

All contributions are obviously welcome. Please include as many details as possible in your PR description to help the reviewer (follow the provided template). Make sure to highlight changes which may need additional attention or you are uncertain about. Any idea with a large scale impact on the crate or its users should ideally be discussed in a "Feature Request" issue beforehand.

Continuous Integration

We use Github Actions for the CI where we perform the following checks:

  • The code should compile on stable and the Minimum Supported Rust Version (MSRV).
  • The tests (docs, lib, tests and examples) should pass.
  • The code should conform to the default format enforced by rustfmt.
  • The code should not contain common style issues clippy.

You can also check most of those things yourself locally using cargo make ci which will offer you a shorter feedback loop.

Tests

The test coverage of the crate is far from being ideal but we already have a fair amount of tests in place. Beside the usual doc and unit tests, one of the most valuable test you can write for tui is a test again the TestBackend. It allows you to assert the content of the output buffer that would have been flushed to the terminal after a given draw call. See widgets_block_renders in tests/widgets_block.rs for an example.