Skip to content

Latest commit

 

History

History
78 lines (48 loc) · 3.28 KB

CONTRIBUTING.md

File metadata and controls

78 lines (48 loc) · 3.28 KB

Contributing

The OQS core team welcomes all proposals to improve this project. This may take the form of a discussion for input or feedback, possible bug reports or feature requests via issues as well as new code and documentation via a pull request (PR).

Review and Feedback

We aim to provide timely feedback to any input. If you are uncertain as to whether a particular contribution is welcome, needed or timely, please first open an issue particularly in case of possible bugs or new feature requests or create a discussion.

Pull requests

Pull requests should clearly state their purpose, possibly referencing an existing issue when resolving it.

All PRs should move to "Ready for Review" stage only if all CI tests pass (are green).

The OQS core team is happy to provide feedback also to Draft PRs in order to improve them before the final "Review" stage.

Coding style

This project has adopted the OpenSSL coding style. To check adherence of any new code to this, it therefore is highly recommended to run the following command in the project main directory prior to finishing a PR:

find oqsprov -type f -and '(' -name '*.h' -or -name '*.c' -or -name '*.inc' ')' | xargs clang-format --dry-run --Werror

Running CI locally

CircleCI

If encountering CI errors in CircleCI, it may be helpful to execute the test jobs locally to debug. This can be facilitated by executing the command

circleci local execute [--job] some-test-job

assuming "some-test-job" is the name of the test to be executed and the CircleCI command line tools have been installed.

Github CI

Act is a tool facilitating local execution of github CI jobs. When executed in the main oqsprovider directory,

act -l Displays all github CI jobs
act -j some-job Executes "some-job"

When installing act as a github extension, prefix the commands with gh .

New features

Any PR introducing a new feature is expected to contain a test of this feature and this test should be part of the CI pipeline, preferably using Github CI.

Background knowledge

New contributors are recommended to first check out documentation of the OpenSSL provider concept as well as the baseline API of liboqs which are the two core foundations for this project.

Failsafe

If you feel your contribution is not getting proper attention, please be sure to add a tag to one or more of our most active contributors.

Issues to start working on

If you feel like contributing but don't know what specific topic to work on, please check the open issues tagged "good first issue" or "help wanted".