mirror of
https://github.com/wez/wezterm.git
synced 2024-09-11 14:25:57 +03:00
* Update CONTRIBUTING.md * Apply suggestions from code review Co-authored-by: Wez Furlong <wez@wezfurlong.org> * Update with new build method. --------- Co-authored-by: Wez Furlong <wez@wezfurlong.org>
132 lines
4.5 KiB
Markdown
132 lines
4.5 KiB
Markdown
# Contributing to wezterm
|
|
|
|
Thanks for considering donating your time and energy! I value any contribution,
|
|
even if it is just to highlight a typo.
|
|
|
|
Included here are some guidelines that can help streamline taking in your contribution.
|
|
They are just guidelines and not hard-and-fast rules. Things will probably go faster
|
|
and smoother if you have the time and energy to read and follow these suggestions.
|
|
|
|
## Contributing Documentation
|
|
|
|
There's never enough! Pretty much anything is fair game to improve here.
|
|
|
|
### Running the doc build yourself
|
|
|
|
To check your documentation additions, you can optionally build the docs yourself and see how the changes will look on the webpage.
|
|
|
|
To serve them up, and then automatically rebuild and refresh the docs in your browser, run:
|
|
```console
|
|
$ ci/build-docs.sh serve
|
|
```
|
|
And then click on the URL that it prints out after it has performed the first build.
|
|
|
|
Any arguments passed to `build-docs.sh` are passed down to the underlying `mkdocs` utility.
|
|
|
|
Look at [mkdocs serve](https://www.mkdocs.org/user-guide/cli/#mkdocs-serve) for more information on additional parameters.
|
|
|
|
### Operating system specific installation instructions?
|
|
|
|
There are a lot of targets out there. Today we have docs that are Ubuntu biased
|
|
and I know that there are a lot flavors of Linux. Rather than expand the README
|
|
with instructions for those, please translate the instructions into steps that
|
|
can be run in the [`get-deps`](https://github.com/wez/wezterm/blob/master/get-deps)
|
|
script.
|
|
|
|
## Contributing code
|
|
|
|
Yes please!
|
|
|
|
If you are new to the Rust language check out <https://doc.rust-lang.org/rust-by-example/>.
|
|
|
|
### Where to find things?
|
|
|
|
The `term` directory holds the core terminal model code. This is agnostic
|
|
of any windowing system. If you want to add support for terminal escape
|
|
sequences and that sort of thing, you probably want to be in the `term` dir.
|
|
Keep in mind that for maximal compatibility and utility `wezterm` aims to
|
|
be compatible with the `xterm` behavior.
|
|
https://invisible-island.net/xterm/ctlseqs/ctlseqs.html is a useful resource!
|
|
|
|
The `src` directory holds the code for the `wezterm` program. This is
|
|
the GUI renderer for the terminal model. If you want to change something
|
|
about the GUI you want to be in the `src` dir.
|
|
|
|
### Iterating
|
|
|
|
I tend to iterate and sanity check as I develop using `cargo check`; it
|
|
will type-check your code without generating code which is much faster
|
|
than building everything in release mode:
|
|
|
|
```console
|
|
$ cargo check
|
|
```
|
|
|
|
Likewise, if you want to quick check that something works, you can run it
|
|
in debug mode using:
|
|
|
|
```console
|
|
$ cargo run
|
|
```
|
|
|
|
This will produce a debug instrumented binary with poor optimization. This will
|
|
give you more detail in the backtrace produced if you run `RUST_BACKTRACE=1 cargo run`.
|
|
|
|
If you get a panic and want to examine local variables, you'll need to use gdb:
|
|
|
|
```console
|
|
$ cargo build
|
|
$ gdb ./target/debug/wezterm
|
|
$ break rust_panic # hit tab to complete the name of the panic symbol!
|
|
$ run
|
|
$ bt
|
|
```
|
|
|
|
### Please include tests to cover your changes!
|
|
|
|
This will help ensure that your contributions keep working as things change.
|
|
|
|
You can run the existing tests using:
|
|
|
|
```console
|
|
$ cargo test --all
|
|
```
|
|
|
|
There are some helper classes for writing tests for terminal behavior.
|
|
Here's an example of a test to verify that the terminal contents
|
|
match expectations:
|
|
|
|
https://github.com/wez/wezterm/blob/master/term/src/test/mod.rs#L334
|
|
|
|
Please also make a point of adding comments to your tests to help
|
|
clarify the intent of the test!
|
|
|
|
### Please also include documentation if you are adding or changing behavior
|
|
|
|
This helps to keep things well-understood and working in the long term.
|
|
Don't worry if you're not a wordsmith or English isn't your first language as
|
|
I can help with that. It is more important to capture the intent of the
|
|
feature and having this written out in English also helps when it comes
|
|
to reviewing the code.
|
|
|
|
## Submitting a Pull Request
|
|
|
|
After considering all of the above, and once you're prepared your contribution
|
|
and are ready to submit it, you'll need to create a pull request.
|
|
|
|
If you're new to GitHub pull requests, read through
|
|
https://help.github.com/articles/creating-a-pull-request/ to understand
|
|
how that process works.
|
|
|
|
### Before your submit your code
|
|
|
|
Make sure that the tests are working and that the code is correctly
|
|
formatted otherwise the continuous integration system will fail your build:
|
|
|
|
```console
|
|
$ rustup component add rustfmt-preview # you only need to do this once
|
|
$ cargo test --all
|
|
$ cargo fmt --all
|
|
```
|
|
|