Skip to content
Snippets Groups Projects

Replace deprecated 'only:' in CI with newer 'rules:'

Merged Replace deprecated 'only:' in CI with newer 'rules:'
ci/gitlab-ci-rules into master
1 unresolved thread
Merged Thomas Kluyver requested to merge ci/gitlab-ci-rules into master
1 unresolved thread

See if this makes the CI run.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
.gitlab-ci.yml
1 workflow:
2 rules:
3 - if: $CI_COMMIT_BRANCH == $CI_DEFAULT_BRANCH
  • Thomas Kluyver
    Thomas Kluyver @kluyvert ·
    Author Owner

    This also means the main test job will run on master when we merge MRs. I don't have a strong feeling about that, but that's a fairly common pattern we use on other projects.

  • Please register or sign in to reply
  • Thomas Kluyver marked this merge request as ready

    marked this merge request as ready

  • Thomas Kluyver
    Thomas Kluyver @kluyvert ·
    Author Owner

    This seems to work, so I'm going to merge it.

  • Thomas Kluyver merged

    merged

  • Thomas Kluyver
    Thomas Kluyver @kluyvert ·
    Author Owner

    Didn't actually make a difference. There might just be something odd about !1026 (merged).

  • Philipp Schmidt changed milestone to %3.16.0

    changed milestone to %3.16.0

  • Thomas Kluyver mentioned in commit ee7c81b6

    mentioned in commit ee7c81b6

  • Philipp Schmidt
    Philipp Schmidt @schmidtp ·
    Owner

    Apparently this was related to the Gitlab issues experienced since the beginning of this year.

    • Please register or sign in to reply

    Legal Notice | Contacts