Skip to content

Commit

Permalink
Merge pull request #207 from MarDiehl/merge-to-experimental-requires-…
Browse files Browse the repository at this point in the history
…tests

explicitly state that unit tests are required
  • Loading branch information
milancurcic authored Jun 4, 2020
2 parents ff99e00 + 1f8681c commit f754a3d
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion WORKFLOW.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,9 @@ experienced contributors will help it through all 5 steps.
people involved in prior steps of the workflow. Other contributors (not
explicitly invited) are encouraged to provide reviews and suggestions as
well. Iterate until all (or most) participants are on the same page.
We can merge when there is vast majority approval of the PR.
A merge is permitted if there are unit tests for a majority of the possible
calling scenarios (with or without optional arguments, with arguments that
trigger an error) and if there is vast majority approval of the PR.

5. **Release**: Moving from experimental to release. The experimental
"namespace" contains new functionality together with its specification. In
Expand Down

0 comments on commit f754a3d

Please sign in to comment.