We're using the "optimistic merging" strategy to make contributing simple and enjoyable.
- To propose a change, log an issue.
- Seek consensus on the value of that change.
- Give the community a day or two to provide feedback on your issue.
- If the feedback is mostly positive, move forward.
- If you don't get any feedback after a day or two, move forward.
- Create a pull request to make the change described in the issue.
- The maintainer review the PR to determine whether it is a "correct patch". That means the PR:
- Solves one "identified and agreed problem"
- Clearly explains the problem and the proposed solution
- Builds without errors, warnings, or test failures
- Does not break our Code of Conduct
- If the PR is a "correct patch", the maintainer merges the PR. If not, the maintainer provides feedback.
If the person reviewing the PR has feedback on a change that is a "correct patch", he or she will merge the PR and include that feedback in a new issue.
To learn more, see Why Optimistic Merging Works Better and Collective Code Construction Contract (C4) from Pieter Hintjens, who coined the term "optimistic merging". The Open Practice Library community has discussed this way of working in #208.
This workflow is still being refined and improved. If you have feedback, feel free to get in touch via Gitter.
Open Practice Library is built with Hugo and the Casper theme. To run it locally:
- Check out this repo.
- Install Hugo.
- Install Node.js.
- Run
git submodule update --init --recursive
to download the theme. - Navigate to the top level of the repo.
- Run
npm install
to install the npm packages. - Run
npx grunt-cli lunr-index
to build the search index. - Run
hugo server
to run the site locally.
Alternately, you can run hugo server -D
to preview drafts.
When changes are merged to master, the build.sh
script will run, and the site will be deployed to https://openpracticelibrary.com. The build log shows the status of each build.