Skip to content
Snippets Groups Projects

Remove include_lfs_blobs_in_archive feature flag [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Merged Stan Hu requested to merge sh-drop-include-lfs-feature-flag into master
All threads resolved!

This feature flag has been enabled by the default since GitLab 13.5, so it's time to remove it.

Part of #268409 (closed)

Edited by Ghost User

Merge request reports

Merged results pipeline #312413996 passed

Pipeline: GitLab

#312414084

    Merged results pipeline passed for fab7dd04

    Test coverage 72.16% (12.80%) from 3 jobs
    Approval is optional

    Merged by Etienne BaquéEtienne Baqué 3 years ago (May 31, 2021 4:57pm UTC)

    Merge details

    Pipeline #312477052 passed

    Pipeline passed for 4b88d6a1 on master

    Test coverage 59.28% (12.80%) from 3 jobs
    5 environments impacted.

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • Stan Hu added 82 commits

    added 82 commits

    Compare with previous version

  • Amy Qualls
  • Amy Qualls approved this merge request

    approved this merge request

    • Resolved by Etienne Baqué

      @stanhu I've added one phrasing suggestion. It wasn't part of your original MR, but I couldn't pretend I didn't see it when I was reading through your changes.

      You removed the right lines. I know the pipeline's red for non-docs reasons, but I'm approving from a docs perspective and unassigning myself.

  • Amy Qualls added devopscreate twfinished labels and removed twdoing label

    added devopscreate twfinished labels and removed twdoing label

  • Amy Qualls removed review request for @aqualls

    removed review request for @aqualls

  • Stan Hu added 1 commit

    added 1 commit

    • 76d4e08e - Improve wording on LFS in archives documentation

    Compare with previous version

  • Stan Hu added 40 commits

    added 40 commits

    Compare with previous version

  • Sean Arnold approved this merge request

    approved this merge request

  • Sean Arnold requested review from @ebaque and removed review request for @seanarnold

    requested review from @ebaque and removed review request for @seanarnold

  • Etienne Baqué approved this merge request

    approved this merge request

  • Etienne Baqué resolved all threads

    resolved all threads

  • Etienne Baqué enabled an automatic merge when the pipeline for ed612960 succeeds

    enabled an automatic merge when the pipeline for ed612960 succeeds

  • Etienne Baqué mentioned in commit 4b88d6a1

    mentioned in commit 4b88d6a1

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • 🤖 GitLab Bot 🤖 removed 1 deleted label

    removed 1 deleted label

  • Please register or sign in to reply
    Loading