Skip to content

Latest commit

 

History

History
44 lines (29 loc) · 3.42 KB

CONTRIBUTING.md

File metadata and controls

44 lines (29 loc) · 3.42 KB
Raw

Contributing

We welcome and encourage all contributions to the project. Please read the following guidelines before submitting a pull request.

Code Contributions

For code contributions, we follow several conventions:

  • Please run cargo +nightly fmt before submitting a pull request. We use rustfmt settings that require nightly.
  • Please run cargo +nightly clippy -F all_features -- -W clippy::perf before submitting a pull request. The project has its clippy preferences set here, and we generally apply clippy's suggestions with those preferences unless there is a good reason not to.
    In that case, our practice is to suppress lints for each specific instance with an optional comment so it does not show up again in future clippy runs, e.g:
#[allow(clippy::unused_assignments)]
let mut var_a = String::with_capacity(10); // amortize allocation
  • Ensure you have the latest version of Rust nightly installed (rustup toolchain update nightly), as we use it for clippy and rustfmt. Running cargo +nightly fmt and cargo +nightly clippy may return different results if you are not using the latest nightly version of Rust.
  • We use docopt for command line argument parsing as we fully take advantage of its ability to parse command line arguments from the contiguous, verbose usage text that is at the beginning of each command's source code that more popular libraries like clap or structopt do not offer.
    However, since docopt.rs is unmaintained, we have a fork that will be maintained along with this project. See this discussion thread for more details.
  • unwrap() and expect() are allowed, but there should be an accompanying comment detailing safety
  • TODO: explain testing conventions, and test helpers
  • TODO: explain error handling conventions
  • TODO: explain logging conventions
  • TODO: release practices
  • TODO: explain the various GitHub Action workflows

Recipe Contributions

We also welcome and highly encourage recipe contributions!

The recipes need not be all that complicated or use qsv exclusively (feel free to mix and match qsv with other CLI tools) but they should be useful and not trivial. We also ask that you include a short description of the recipe, and a link to the source of the recipe if it is not your own.

Just go to the Cookbook and add your recipe.

Once you add a recipe, show it in action! Add a "Show and tell" entry in Discussions, and link to your recipe.

Documentation Contributions

We are always looking for ways to improve the documentation. If you find a typo, or have a suggestion for improvement, please submit a pull request.

And if you want to add a new page to the Wiki, please do! Just make sure to add section headers to your Wiki contributions, so it automatically shows up in the sidebar.

If you are not familiar with GitHub Wikis, you can find a guide here.