Skip to content

[BRE] Removing build workflow and Dockerfiles - #8173

Open
gitclonebrian wants to merge 2 commits into
mainfrom
bre/remove-build-workflow
Open

[BRE] Removing build workflow and Dockerfiles#8173
gitclonebrian wants to merge 2 commits into
mainfrom
bre/remove-build-workflow

Conversation

@gitclonebrian

@gitclonebrian gitclonebrian commented Aug 7, 2026

Copy link
Copy Markdown
Member

🎟️ Tracking

BRE-2163

📔 Objective

Removing build workflow and Dockerfiles.

@gitclonebrian
gitclonebrian marked this pull request as ready for review August 7, 2026 19:21
@gitclonebrian
gitclonebrian requested review from a team as code owners August 7, 2026 19:21
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR removes the Build Gov workflow (.github/workflows/build-gov.yml) and the twelve accompanying gov.Dockerfile files across src/, util/, and bitwarden_license/src/. The change is deletion-only with no source, configuration, or test modifications. I verified that no remaining file in the repository references build-gov, gov.Dockerfile, or the bwfedrampdev registry, and that the standard Dockerfile for each project plus the primary build.yml workflow are untouched.

Code Review Details

No findings.

Verification performed:

  • Searched .github/ and the full tree for gov / gov.Dockerfile / build-gov references — none remain outside unrelated matches in src/Icons/Resources/public_suffix_list.dat and the fedramp-gov-region feature flag constant in src/Core/Constants.cs, neither of which depends on the deleted files.
  • Confirmed .github/renovate.json5 dockerfile managers and .github/CODEOWNERS Dockerfile globs are pattern-based and need no update.
  • Confirmed all non-gov Dockerfile files and .github/workflows/build.yml remain intact.

Note for the author: if Build Gov was configured as a required status check on any protected branch, that branch protection rule will need to be removed separately since it lives outside the repository.

@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.06%. Comparing base (9a952ca) to head (d66c5a0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8173   +/-   ##
=======================================
  Coverage   63.06%   63.06%           
=======================================
  Files        2319     2319           
  Lines      100593   100593           
  Branches     9057     9057           
=======================================
  Hits        63442    63442           
  Misses      34962    34962           
  Partials     2189     2189           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gitclonebrian gitclonebrian added the t:ci Change Type - Updates to automated workflows label Aug 7, 2026
@gitclonebrian
gitclonebrian enabled auto-merge (squash) August 7, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:ci Change Type - Updates to automated workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant