Skip to content
Snippets Groups Projects

Enable ci_needs_optional by default

Merged Furkan Ayhan requested to merge 323891-enable-ci_needs_optional into master
All threads resolved!

What does this MR do?

This MR enables ci_needs_optional by default.

It's enabled on GitLab.com => #323891 (comment 538462483)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Furkan Ayhan

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
  • requested review from @marcel.amirault

  • Marcel Amirault
  • Marcel Amirault approved this merge request

    approved this merge request

  • Marcel Amirault removed review request for @marcel.amirault

    removed review request for @marcel.amirault

  • Furkan Ayhan added 1 commit

    added 1 commit

    • 04a0c3fa - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Furkan Ayhan resolved all threads

    resolved all threads

  • Furkan Ayhan changed the description

    changed the description

  • Author Maintainer

    @reprazent Roulette did not work for this MR so I directly selected a maintainer from GitLab Review Workload Dashboard. I think we can assign this kind of MRs directly to maintainers 🤔

    Edited by Furkan Ayhan
  • Furkan Ayhan requested review from @reprazent

    requested review from @reprazent

  • Bob Van Landuyt approved this merge request

    approved this merge request

  • I think we can assign this kind of MRs directly to maintainers 🤔

    @furkanayhan Yep, that's fine!

    The last pipeline completed less than an hour ago. Merging manually.

  • Bob Van Landuyt mentioned in commit ae3776b8

    mentioned in commit ae3776b8

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading