Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • mesa mesa
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3,083
    • Issues 3,083
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1,020
    • Merge requests 1,020
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

  • MesaMesa
  • mesamesa
  • Merge requests
  • !1960

gitlab-ci: Report unexpected passes as failures

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Emma Anholt requested to merge anholt/mesa:unexpected-passes into master Sep 12, 2019
  • Overview 5
  • Commits 3
  • Pipelines 2
  • Changes 2

@krh asked if CI would do this, and I thought it was reasonable behavior, so here goes:

If people fix bugs without updating the expected-fails list, then we end up with a lack of coverage of those failures in the future, and some day down the line another developer ends up trying to figure out if the bug was actually fixed or their environment is just failing to reproduce it.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: unexpected-passes