Pull Request Requirements

Whenever a pull request is submitted, there are some expectations for the content before it can be merged into the master branch.

Code changes

There isn't any strict rule on how coding should be done for PyPDFForm. The project welcomes code contributions from anyone with any level of expertise.

With that said, there are some conventions that are expected to be followed by your PR:

  • Your changes must pass Pylint. To check if this is true, simply run pylint PyPDFForm.
  • Your changes must pass all tests and have 100% coverage. You can read more about testing here.
  • If you are changing the user APIs or any other parts of the code that are relevant, please update the appropriate documentation too.

Merge process

Your PR will be reviewed before merging into the master branch. On top of that, it needs to run through some CI checks:

  • Pylint on the source code.
  • Tests will be run on three mainstream operating systems: ubuntu, windows, and macos, and across all Python versions the library supports on each OS.

Once the CI is green and your code looks good, the PR will be merged into the master branch. After every PR merge, black and isort will be run on your code, and they will be deployed on the next release.