Skip to content

Remove sbom and scan plugin packages - #254

Merged
thaJeztah merged 1 commit into
docker:mainfrom
vvoland:remove-scan
Sep 3, 2025
Merged

Remove sbom and scan plugin packages#254
thaJeztah merged 1 commit into
docker:mainfrom
vvoland:remove-scan

Conversation

@vvoland

@vvoland vvoland commented Sep 2, 2025

Copy link
Copy Markdown
Collaborator

They are no longer supported

They are no longer supported

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the sbom and scan plugin packages from the Docker packaging repository as they are no longer supported. This cleanup effort eliminates all build configurations, Docker files, packaging scripts, and GitHub workflows related to these deprecated plugins.

  • Removes entire pkg/scan/ and pkg/sbom/ directories containing all plugin-related files
  • Updates build configuration to exclude these packages from the build matrix
  • Removes GitHub workflow files for building and releasing these plugins

Reviewed Changes

Copilot reviewed 28 out of 28 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/scan/ Complete removal of Docker scan plugin packaging files including Dockerfiles, build scripts, and package specs
pkg/sbom/ Complete removal of Docker SBOM plugin packaging files including Dockerfiles, build scripts, and package specs
docker-bake.hcl Removes scan and sbom from PKGS list and removes their build target configurations
.github/workflows/ Removes build and release workflows for scan and sbom plugins
.github/labeler.yml Removes labeler rules for scan and sbom package areas
.github/workflows/nightly.yml Removes DISABLED_PACKAGES environment variable that was excluding scan and sbom

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@vvoland

vvoland commented Sep 3, 2025

Copy link
Copy Markdown
Collaborator Author

@crazy-max PTAL

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
thaJeztah merged commit ef69817 into docker:main Sep 3, 2025
416 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants