Skip to content

Run Hadolint on all files, not just changed ones#9286

Merged
SpecLad merged 2 commits intocvat-ai:developfrom
SpecLad:hadolint-all-files
Mar 31, 2025
Merged

Run Hadolint on all files, not just changed ones#9286
SpecLad merged 2 commits intocvat-ai:developfrom
SpecLad:hadolint-all-files

Conversation

@SpecLad
Copy link
Contributor

@SpecLad SpecLad commented Mar 28, 2025

Motivation and context

Same reasoning as #5720.

How has this been tested?

Checklist

  • I submit my changes into the develop branch
  • [ ] I have created a changelog fragment
  • [ ] I have updated the documentation accordingly
  • [ ] I have added tests to cover my changes
  • [ ] I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@SpecLad SpecLad force-pushed the hadolint-all-files branch from 290174c to 327ffbf Compare March 28, 2025 18:22
@SpecLad SpecLad requested a review from azhavoro as a code owner March 28, 2025 18:25
@sonarqubecloud
Copy link

@SpecLad SpecLad merged commit 273a3ba into cvat-ai:develop Mar 31, 2025
34 checks passed
@SpecLad SpecLad deleted the hadolint-all-files branch March 31, 2025 09:58
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.17%. Comparing base (9f4a467) to head (91cbf27).
Report is 9 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9286      +/-   ##
===========================================
+ Coverage    74.15%   74.17%   +0.02%     
===========================================
  Files          428      428              
  Lines        44967    44964       -3     
  Branches      3917     3918       +1     
===========================================
+ Hits         33344    33352       +8     
+ Misses       11623    11612      -11     
Components Coverage Δ
cvat-ui 77.23% <ø> (+0.05%) ⬆️
cvat-server 71.63% <ø> (+<0.01%) ⬆️
🚀 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments