Skip to content

Latest commit

 

History

History
134 lines (83 loc) · 9.59 KB

File metadata and controls

134 lines (83 loc) · 9.59 KB

Contributing to WP Document Revisions

Hi there! We're thrilled that you'd like to contribute to WP Document Revisions. Your help is essential for keeping it great.

WP Document Revisions is an open source project supported by the efforts of an entire community and built one contribution at a time by users like you. We'd love for you to get involved. Whatever your level of skill or however much time you can give, your contribution is greatly appreciated. There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests, helping other users by commenting on issues, or writing code which can be incorporated into WP Document Revisions itself.

Following these guidelines helps to communicate that you respect the time of the developers managing and developing this open source project. In return, they should reciprocate that respect in addressing your issue, assessing changes, and helping you finalize your pull requests.

Looking for support?

We'd love to help. Check out the support guidelines.

How to report a bug

Think you found a bug? Please check the list of open issues to see if your bug has already been reported. If it hasn't please submit a new issue.

Here are a few tips for writing great bug reports:

  • Describe the specific problem (e.g., "widget doesn't turn clockwise" versus "getting an error")
  • Include the steps to reproduce the bug, what you expected to happen, and what happened instead
  • Check that you are using the latest version of the project and its dependencies
  • Include what version of the project your using, as well as any relevant dependencies
  • Only include one bug per issue. If you have discovered two bugs, please file two issues
  • Include screenshots or screencasts whenever possible
  • Even if you don't know how to fix the bug, including a failing test may help others track it down

If you find a security vulnerability, do not open an issue. Please email ben@balter.com instead.

How to suggest a feature or enhancement

If you find yourself wishing for a feature that doesn't exist in WP Document Revisions, you are probably not alone. There are bound to be others out there with similar needs. Many of the features that WP Document Revisions has today have been added because our users saw the need.

Feature requests are welcome. But take a moment to find out whether your idea fits with the scope and goals of the project. It's up to you to make a strong case to convince the project's developers of the merits of this feature. Please provide as much detail and context as possible, including describing the problem you're trying to solve.

Open an issue which describes the feature you would like to see, why you want it, how it should work, etc.

Ways to Contribute

Your first contribution

We'd love for you to contribute to the project. Unsure where to begin contributing to WP Document Revisions? You can start by looking through these "good first issue" and "help wanted" issues:

  • Good first issues - issues which should only require a few lines of code and a test or two
  • Help wanted issues - issues which may be a bit more involved, but are specifically seeking community contributions

p.s. Feel free to ask for help; everyone is a beginner at first 😺

How to propose changes

Here's a few general guidelines for proposing changes:

  • If you are changing any user-facing functionality, please be sure to update the documentation
  • If you are adding a new behavior or changing an existing behavior, please be sure to update the corresponding test(s)
  • Each pull request should implement one feature or bug fix. If you want to add or fix more than one thing, submit more than one pull request
  • Do not commit changes to files that are irrelevant to your feature or bug fix
  • Don't bump the version number in your pull request (it will be bumped prior to release)
  • Write a good commit message

At a high level, the process for proposing changes is:

  1. Fork and clone the project
  2. Configure and install the dependencies: script/bootstrap
  3. Make sure the tests pass on your machine: script/cibuild
  4. Create a descriptively named branch: git checkout -b my-branch-name
  5. Make your change, add tests and documentation, and make sure the tests still pass
  6. Push to your fork and submit a pull request describing your change
  7. Pat your self on the back and wait for your pull request to be reviewed and merged

Interesting in submitting your first Pull Request? It's easy! You can learn how from this free series How to Contribute to an Open Source Project on GitHub

Bootstrapping your local development environment

script/bootstrap

Running tests

script/cibuild

Production composer dependencies

The plugin uses smalot/pdfparser for PDF text extraction (see issue #514). It is installed via composer install --no-dev and ships unscoped from vendor/ to wordpress.org today.

Dev workflow

A plain composer install is enough — both dev tools and production deps land in vendor/, and wp-document-revisions.php boots from vendor/autoload.php.

Release workflow

Pushing any tag deploys to WordPress.org, so tag a release only after the owner approves it.

The deploy workflow (.github/workflows/deploy.yml) runs composer install --no-dev before invoking the WordPress.org deploy action so the release artifact contains only production deps in vendor/. .distignore is set up so vendor/ ships and composer.json / composer.lock do not.

Optional: scoped vendor (work in progress)

A php-scoper pipeline (composer build:scope, scoper.inc.php, vendor-bin/scoper/) is in place so that production composer deps can be rewritten under WP_Document_Revisions\Vendor\ and avoid namespace collisions with the same library shipped by another wordpress.org plugin. The pipeline runs in CI as a smoke check but is not on the release path yet — php-scoper's handling of Composer's own autoload bootstrap needs more work before the scoped output is safe to load. When that work lands, wp-document-revisions.php already prefers vendor-prefixed/ when it exists, and the deploy workflow will switch to running composer build:scope after the production install.

Continuous Integration and Security

The project uses GitHub Actions for continuous integration and automated security scanning:

  • PHPUnit Tests: Run on multiple PHP versions (7.2-8.3) and WordPress versions
  • PHPCS: WordPress coding standards enforcement
  • Jest: JavaScript unit tests
  • CodeQL: Automated security scanning for JavaScript code

CodeQL is configured to analyze only JavaScript since CodeQL does not yet support PHP. The configuration files are:

  • .github/workflows/codeql.yml - Defines the CodeQL analysis workflow
  • .github/codeql-config.yml - Specifies paths to analyze and exclude

Ruby is explicitly excluded from analysis as this repository contains no Ruby code.

Code of conduct

This project is governed by the Contributor Covenant Code of Conduct. By participating, you are expected to uphold this code.

Developing with Docker

Prefer to use Docker to develop locally?

  1. git clone https://github.com/wp-document-revisions/wp-document-revisions/
  2. cd wp-document-revisions
  3. docker-compose up
  4. open http://localhost:8088

Additional Resources