FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

pre-commit: auto add license headers for all YAML files by jbampton · Pull Request #12069 · apache/cloudstack · GitHub

  1. apache
  2. cloudstack

Repository navigation

pre-commit: auto add license headers for all YAML files - #12069

Merged
DaanHoogland merged 2 commits into
apache:mainfrom
jbampton:add-license-headers-yaml
Nov 14, 2025
ConversationCommits (2)ChecksFiles changed
Merged

pre-commit: auto add license headers for all YAML files#12069
DaanHoogland merged 2 commits into
apache:mainfrom
jbampton:add-license-headers-yaml

Conversation

jbampton commented Nov 13, 2025

Copy link
Copy Markdown
Member

Fix and standardize one license header

Description

This PR uses pre-commit to auto add license headers to YAML files when they are missing.

So one step better that just checking for missing licenses with the rat pom check.

We are already doing this for Markdown files.

A bit more automation added

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • Build/CI
  • Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

With pre-commit locally

How did you try to break this feature and the system with this change?

boring-cyborg Bot added the docker label Nov 13, 2025

codecov Bot commented Nov 13, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 17.56%. Comparing base (86ae1fe) to head (ee081e4).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff            @@
##               main   #12069   +/-   ##
=========================================
  Coverage     17.56%   17.56%           
- Complexity    15539    15540    +1     
=========================================
  Files          5911     5911           
  Lines        529359   529359           
  Branches      64655    64655           
=========================================
+ Hits          92979    92981    +2     
+ Misses       425920   425919    -1     
+ Partials      10460    10459    -1
Flag Coverage Δ
uitests 3.58% <ø> (ø)
unittests 18.63% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

jbampton commented Nov 13, 2025

Copy link
Copy Markdown
Member Author

@blueorangutan package

blueorangutan commented Nov 13, 2025

Copy link
Copy Markdown

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

This was referenced Nov 13, 2025
Merged
Merged

blueorangutan commented Nov 13, 2025

Copy link
Copy Markdown

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15754

DaanHoogland merged commit aa18188 into apache:main Nov 14, 2025
28 checks passed
jbampton deleted the add-license-headers-yaml branch November 14, 2025 13:40
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Dec 11, 2025
pre-commit: auto add license headers for all YAML files (apache#12069)
83c6e0a 
Fix and standardize one license header
owsferraro pushed a commit to owsferraro/cloudstack that referenced this pull request Mar 25, 2026
pre-commit: auto add license headers for all YAML files (apache#12069)
0b2bb89 
Fix and standardize one license header
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

DaanHoogland DaanHoogland approved these changes

Assignees

No one assigned

Labels

docker

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL