oblachno-oss/devx
Public Access
2
0
Fork 0
Code Issues 36 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

DEVX-83: Make hadolint mandatory in lint-dockerfiles #133

Merged
emil merged 1 commits from DEVX-83-hadolint-fail-fast into master 2026-06-27 16:43:23 +00:00
Conversation 3 Commits 1 Files Changed 3
+50 -7
emil commented 2026-06-27 16:27:07 +00:00
Owner
Copy Link
Copy Source
No description provided.
emil reviewed 2026-06-27 16:27:18 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: OK
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil force-pushed DEVX-83-hadolint-fail-fast from eb972843fa to 6c9021d532 2026-06-27 16:33:43 +00:00 Compare
emil reviewed 2026-06-27 16:33:56 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil added 1 commit 2026-06-27 16:42:21 +00:00
fix: fail lint-dockerfiles when hadolint is missing
CI / detect-changes (pull_request) Successful in 11s
Details
CI / pr-review (pull_request) Successful in 11s
Details
CI / quality (pull_request) Successful in 35s
Details
CI / release-dry-run (pull_request) Successful in 11s
Details
CI / auto-merge (pull_request) Successful in 12s
Details
dd0691729a 
Silently skipping a missing linter hides issues. Fail early so
the tool must be installed for lint checks to pass.
emil force-pushed DEVX-83-hadolint-fail-fast from 6c9021d532 to dd0691729a 2026-06-27 16:42:21 +00:00 Compare
emil reviewed 2026-06-27 16:42:33 +00:00
emil left a comment
Author
Owner
Copy Link
Copy Source

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil merged commit 233a0bc055 into master 2026-06-27 16:43:23 +00:00
emil deleted branch DEVX-83-hadolint-fail-fast 2026-06-27 16:43:23 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
emil
Labels
Clear labels
bug
Something is not working
 ci-improvement
CI workflow improvements
 doc-improvement
Documentation improvements
 feedback
Issues from subagent or agent feedback
 ready-to-merge
tooling
Tool-related feedback or improvements
 workflow-improvement
Workflow alignment or pattern improvements
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
emil (emil User) kireto
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#133