search

opensearch-project / opensearch-metrics

OpenSearch Metrics
https://metrics.opensearch.org
Apache License 2.0
2 stars 4 forks source link

[BUG] Missing license header #55

Open gaiksaya opened 2 months ago

gaiksaya commented 2 months ago

What is the bug?

Many files in this repo are missing license headers

How can one reproduce the bug?

Go through https://github.com/opensearch-project/opensearch-metrics/tree/main/src

What is the expected behavior?

All files should have license headers. Also as an enhancement, add license checker similar to https://github.com/opensearch-project/opensearch-build/blob/main/.github/workflows/license-header-checker.yml

What is your host/environment?

Operating system, version.

Do you have any screenshots?

If applicable, add screenshots to help explain your problem.

Do you have any additional context?

Add any other context about the problem.

prudhvigodithi commented 2 months ago

[Triage] Thanks for the PR https://github.com/opensearch-project/opensearch-metrics/pull/54 @gaiksaya @getsaurabh02 @bshien

prudhvigodithi commented 1 month ago

@gaiksaya can we close this issue?

gaiksaya commented 1 month ago

@gaiksaya can we close this issue?

I dont think so. I just added for infrastructure code but java code still needs license headers. Regarding license checker I think that can be tracked in separate issue.