bottom/CONTRIBUTING.md
Clement Tsang 6db76029e2
feature: Beginnings of in-app config (#231)
Initial refactorings and additions to support in-app config.

- Refactor our current options logic to support in-app configs.  That is, we can write to a config file with our changes now.
- The default action when creating a new config file is to leave it blank. (TBD and for now, not sure on this one)
- Previously, we would set everything in a config file on startup; now we need to read from the config TOML struct whenever.
- `C` keybind is now occupied for configs.
- `no_write` option to never write to a config file.
2020-09-22 18:12:36 -04:00

4.4 KiB

Contribution

If you want to contribute to this project - first of all, thank you! I'm glad to see interest in it!

Here are some notes about how to contribute to bottom (structure is based on the official Rust contribution guidelines):

Feature reports

Feature suggestions can be submitted using the "feature" tag. Prior to submission, please look to see if this has already been suggested or solved; if it has and is not resolved, it would be better to comment on the relevant report.

Within your feature report, try to answer the given prompts - in particular, state the specific feature you want and if possible, please state why you want this added to the program.

Bug reports

Bug reports can be submitted using the "bug" tag. Prior to submission, please look to see if this has already been reported or solved; if it has and is not resolved, it would be better to comment on the relevant report.

Within your bug report, try to answer the given prompts. Be as specific as possible - describe your bug to the best of your ability, how to replicate it, and provide information like screenshots, OS and terminal. It can be very useful to help whoever is dealing with the issue!

Other types of reports

For reports/suggestions that don't fit the definition of a feature or bug, try to use the other tags:

  • documentation: If you note a typo, or want to suggest something to do with the documentation of bottom, use this.
  • question: If you have a question, and not really a suggestion or request, then use this tag.
  • ci, investigative, refactoring: Generally, these are for internal use to track issues in order to manage GitHub Projects, and won't be the appropriate topic for a report.
  • other: If your suggestion or issue doesn't fit those categories, then use this.

Pull requests

If you want to help contribute by submitting a PR, by all means, I'm open! In regards to the development process:

  • I develop primarily using stable Rust. That is, whatever is the most up-to-date stable version you can get via running rustup update stable.

  • There are some tests, they're mostly for sanity checks. Please run cargo test to ensure you didn't break anything important, unless the change will break the test (in which case please amend the tests).

    • Note that cargo test will fail on anything lower than 1.43.0 due to it using a then-introduced env variable.
  • I use both clippy and rustfmt in development (with some settings, see clippy.toml and rustfmt.toml). Note clippy must pass to for PRs to be accepted.

    • You can check clippy using cargo clippy.

    • I use cargo-husky to automatically run a cargo clippy check.

  • You may notice that I have fern and log as dependencies; this is mostly for easy debugging via the debug!() macro. It writes to the debug.log file that will automatically be created if you run in debug mode (so cargo run).

And in regards to the pull request process:

  • Create a personal fork of the process and PR that, as per the fork and pull method.

  • Merge against the master branch.

  • If you encounter a merge conflict, I expect you to resolve this via rebasing to master.

  • Ensure your change builds, runs, and works. Furthermore, state how you checked this, including what platforms you tested on.

  • If your change will result in needing to update documentation, please do so. In particular:

    • Does the README need to be updated to accommodate your change?

    • Does the help action, ?, need to be updated to accommodate your change?

  • Please ensure that CI passes. If it fails, check to see why it fails! Chances are it's clippy.

  • If all looks good, then request someone with write access (so basically me, @ClementTsang) to give your code a review. If it's fine, then I'll merge!

  • Please use the pull request template to help you.