Skip to content

Commit 6c06dfa

Browse files
committed
Deploy preview for PR 635
1 parent aca9561 commit 6c06dfa

File tree

3 files changed

+47
-47
lines changed

3 files changed

+47
-47
lines changed

pr-previews/635/developers/contributing/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -794,7 +794,7 @@ <h4 class="anchored" data-anchor-id="continuous-integration-ci-tests">Continuous
794794
<li>Anything running on a prerelease of Julia. These inform us of trouble ahead when that prerelease becomes an actual release, but don’t require fixing for a PR to be merged.</li>
795795
<li>Any code coverage checks. Code coverage numbers can be helpful in catching missing tests or cases where the tests don’t test what they are intended to. However, we do not insist on any particular coverage figures, since they are not a very good metric of a test suite’s extensiveness.</li>
796796
<li>The benchmarks on DynamicPPL repo. These should be investigated to understand why they fail. If the reason is a bug in the PR, an actual test should be added to the test suite to catch it. However, sometimes they fail for unrelated reasons.</li>
797-
<li>Checks against some particular automatic differentiation backends. These are informative, telling us which backends have issues with which features, but often their breakage is beyond our control and we let it pass.</li>
797+
<li>Occasionally CI failures are caused by bugs that require upstream fixes (such as for AD backends, or base Julia). Please ping a maintainer if you are unsure if this is the case. A good indicator for this is if the same test is failing on the base branch of your pull request.</li>
798798
<li>The CI check in the <code>docs</code> repo for whether the docs are built with the latest Turing.jl release. This test failing is a reminder that we should make a PR to update to the latest version, but does not need fixing when working on a PR that makes unrelated changes to the documentation.</li>
799799
</ul>
800800
<p>If you are ever unsure whether some CI check needs to pass, or if the reason why one is failing is mysterious or seems unrelated to the PR, ask a maintainer and they’ll help you out.</p>

0 commit comments

Comments
 (0)