Skip to content

ci: add hadolint docker pre-commit config - #994

Merged
jbampton merged 2 commits into
brisbanesocialchess:mainfrom
Subham-KRLX:add-hadolint-docker
Feb 28, 2026
Merged

jbampton merged 2 commits into
brisbanesocialchess:mainfrom
Subham-KRLX:add-hadolint-docker

Conversation

@Subham-KRLX

@Subham-KRLX Subham-KRLX commented Feb 28, 2026

Copy link
Copy Markdown
Contributor

Adds a custom .pre-commit-docker-config.yaml file containing the hadolint-docker hook to lint Dockerfiles.

Fixes #990

Pull Request

PR Checklist

  • I have tested my changes locally.
  • I have run pre-commit run --all-files locally.
  • I have updated the documentation if applicable.
  • I have reviewed my code for any potential issues.

PR Type

  • Feature

Current behavior

No separate pre-commit configuration exists for linting Dockerfiles with the hadolint/hadolint Docker image.

New behavior

Added .pre-commit-docker-config.yaml containing the official hadolint-docker hook (v2.14.0) for dedicated Dockerfile linting.

Adds a custom `.pre-commit-docker-config.yaml` file
containing the hadolint-docker hook to lint Dockerfiles.

Fixes brisbanesocialchess#990
@deepsource-io

deepsource-io Bot commented Feb 28, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 34348f0...8922023 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Docker Feb 28, 2026 7:13p.m. Review ↗
JavaScript Feb 28, 2026 7:13p.m. Review ↗
Python Feb 28, 2026 7:13p.m. Review ↗
Secrets Feb 28, 2026 7:13p.m. Review ↗

@jbampton jbampton added this to Chess Feb 28, 2026
@jbampton jbampton moved this to In Progress in Chess Feb 28, 2026
@jbampton jbampton added this to the 2026 First Quarter milestone Feb 28, 2026
@jbampton jbampton self-assigned this Feb 28, 2026
@github-actions github-actions Bot removed the tests label Feb 28, 2026

@jbampton jbampton left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks 🔢

@jbampton
jbampton merged commit 1e5afe0 into brisbanesocialchess:main Feb 28, 2026
32 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Chess Feb 28, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Chess Feb 28, 2026
Subham-KRLX added a commit to Subham-KRLX/brisbanesocialchess.github.io that referenced this pull request Mar 12, 2026
Adds a custom `.pre-commit-docker-config.yaml` file
containing the hadolint-docker hook to lint Dockerfiles.

Fixes brisbanesocialchess#990

Co-authored-by: John Bampton <jbampton@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add hadolint docker with pre-commit in a new separate custom pre-commit-docker-config.yaml

2 participants