Skip to content

PR with bad license#400

Open
dlvenable wants to merge 4 commits intomainfrom
bad-license-test
Open

PR with bad license#400
dlvenable wants to merge 4 commits intomainfrom
bad-license-test

Conversation

@dlvenable
Copy link
Owner

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.

… One for checking the PR and the other for posting PR comments.

Signed-off-by: David Venable <dlv@amazon.com>
@github-actions
Copy link

github-actions bot commented Mar 12, 2026

Unit Test Results

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

Results for commit 5c368a6. ± Comparison against base commit 433bf50.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Mar 12, 2026

⚠️ License Header Violations Found

The following newly added files are missing required license headers:

  • data-prepper-api/src/test/java/org/opensearch/dataprepper/AnotherNewTest.java
  • data-prepper-api/src/test/java/org/opensearch/dataprepper/NewTest.java

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

Signed-off-by: David Venable <dlv@amazon.com>
Signed-off-by: David Venable <dlv@amazon.com>
Signed-off-by: David Venable <dlv@amazon.com>
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