Skip to content

Add test files for license header compliance testing#397

Closed
dlvenable wants to merge 1 commit into
mainfrom
test-license-headers-stay-bad
Closed

Add test files for license header compliance testing#397
dlvenable wants to merge 1 commit into
mainfrom
test-license-headers-stay-bad

Conversation

@dlvenable

Copy link
Copy Markdown
Owner
  • Good files with complete license headers (4 files):

    • Java: TestProcessor.java
    • Python: setup-environment.py
    • YAML: docker-compose.yml
    • Properties: application.properties
  • Bad files with missing/incomplete headers (5 files):

    • Java: NewFeature.java (no header)
    • Python: monitor.py (short header)
    • YAML: data-prepper-config.yaml (no header)
    • Properties: dev.properties (no header)
    • Shell: compile.sh (incomplete header)

These files test the license header compliance system across all supported file types and various violation scenarios.

Description

[Describe what this change achieves]

Issues Resolved

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
  • New functionality has a documentation issue. Please link to it in this PR.
    • New functionality has javadoc added
  • Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

- Good files with complete license headers (4 files):
  * Java: TestProcessor.java
  * Python: setup-environment.py
  * YAML: docker-compose.yml
  * Properties: application.properties

- Bad files with missing/incomplete headers (5 files):
  * Java: NewFeature.java (no header)
  * Python: monitor.py (short header)
  * YAML: data-prepper-config.yaml (no header)
  * Properties: dev.properties (no header)
  * Shell: compile.sh (incomplete header)

These files test the license header compliance system across
all supported file types and various violation scenarios.
@github-actions

github-actions Bot commented Jan 6, 2026

Copy link
Copy Markdown

⚠️ License Header Violations Found

The following newly added files are missing required license headers:

  • config/environments/dev.properties
  • data-prepper-core/src/main/java/org/opensearch/dataprepper/core/NewFeature.java
  • deployment/kubernetes/data-prepper-config.yaml
  • scripts/compile.sh
  • tools/monitoring/monitor.py

Please add the appropriate license header to each file and push your changes.

See the license header requirements: https://github.com/opensearch-project/data-prepper/blob/main/CONTRIBUTING.md#license-headers

@github-actions

github-actions Bot commented Jan 6, 2026

Copy link
Copy Markdown

Unit Test Results

0 tests  ±0   0 ✔️ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ±0 

Results for commit 46e6aa2. ± Comparison against base commit 8bb1808.

♻️ This comment has been updated with latest results.

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.

1 participant