You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

56 lines
3.1 KiB

5 years ago
  1. # Contributing to ACMEd
  2. ## Testing and bug report
  3. The first way to help is to actually use the software and [report any bug you encounter](https://github.com/breard-r/acmed/issues). Do not hesitate to test the limits.
  4. ## Improving the language
  5. Since the author is not a native English speaker, some of the texts used in this project should be fixed.
  6. ## Work on dependencies
  7. ### attohttpc
  8. Although `attohttpc` is not currently a dependency, it may replace `reqwest` which is far too big and drags a lot of dependencies. But before this could be done, it needs to allow [new root certificates to be added](https://github.com/sbstp/attohttpc/issues/71).
  9. ### rust-openssl
  10. An improvement that would be appreciable is to add Curve 25519 support to the [openssl](https://crates.io/crates/openssl) crate.
  11. - https://github.com/sfackler/rust-openssl/issues/947
  12. - https://github.com/sfackler/rust-openssl/pull/1275
  13. ### Find or create a good template engine
  14. As reported in [issue #8](https://github.com/breard-r/acmed/issues/8), there is currently no perfect template engine. A good way to help improve ACMEd would be to find or create one that supports all the listed requirements.
  15. ## Improving the code
  16. As a one-man project, it has several goals already set but not explicitly written in an issue or any other follow-up file. It will not be the case before version 1.0 is released since everything may change at any moment. Therefore, it is recommended to request change instead of implementing them, this way we can discuss how things should be made.
  17. If you really want to submit a pull request, please :
  18. - document your changes in the man pages and the `CHANGELOG.md` file
  19. - write as much tests as you can
  20. - run `cargo test` and be sure every test pass
  21. - format your code using [rustfmt](https://github.com/rust-lang/rustfmt)
  22. - be sure not to have any warning when compiling
  23. - run [clippy](https://github.com/rust-lang/rust-clippy) and fix any issue
  24. - refrain from including a new dependency (crates having `ring` in their dependency tree are a no-go, see #2)
  25. - beware of potential repercussions on the default hooks: those should remain usable
  26. Not following the rules above will delay the merge since they will have to be fixed first.
  27. ## Author vs. contributor
  28. Some people have troubles seeing the difference between an author and a contributor. Here is how it is seen withing this project.
  29. A contributor is a person who helps the project in various ways whenever she or he wants. As such, a contributor does not have any obligation other than being respectful and open-minded. People who wrote code that have been accepted are automatically listed in the [contributors page](https://github.com/breard-r/acmed/graphs/contributors). The creation of a file with the names of people contributing outside of the code base will be studied upon request from such people.
  30. An author is a person who has some responsibilities on the project. Authors are expected to contribute on a regular basis, decide architectural choices, enforce copyright issues and so on. One does not grant himself the author status, it is given by the others authors after having discussed the request.