Skip to content

Gate PR's so they require CI checks to pass #698

Open
@jeffwidman

Description

@jeffwidman

In #652, @StephenSorriaux very nicely fixed up our CI checks.

However, our current PR's aren't required to pass CI before being merged:

  1. We should change this so they have to pass CI.
  2. We need to retain the ability to override in case for some reason a PR intentionally won't pass CI but we still need to merge it. While very rare, this does occasionally happen. Several of the maintainers have admin rights on the repo so they could temporarily disable those branch checks if needed.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions