Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating license headers #196

Merged
merged 3 commits into from
Nov 17, 2021

Conversation

downsrob
Copy link
Contributor

@downsrob downsrob commented Nov 16, 2021

Signed-off-by: Robert Downs downsrob@amazon.com

Issue #, if available:
#114

Description of changes:
Corrects the copyright license header as per the linked issue requirements. Besides the gradlew file, which maintained the same copyright as an external file, the short form was used for all files, as I didn't notice any others with an attribution to an external project.

CheckList:

  • Commits are signed per the DCO using --signoff

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.

Sorry, something went wrong.

Signed-off-by: Robert Downs <downsrob@amazon.com>
Signed-off-by: Robert Downs <downsrob@amazon.com>
@codecov-commenter
Copy link

Codecov Report

Merging #196 (512c408) into main (1fe36be) will decrease coverage by 0.20%.
The diff coverage is n/a.

❗ Current head 512c408 differs from pull request most recent head 0e3361e. Consider uploading reports for the commit 0e3361e to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##               main     #196      +/-   ##
============================================
- Coverage     75.69%   75.49%   -0.21%     
+ Complexity     1926     1920       -6     
============================================
  Files           260      260              
  Lines         11094    11094              
  Branches       1746     1746              
============================================
- Hits           8398     8375      -23     
- Misses         1755     1774      +19     
- Partials        941      945       +4     
Impacted Files Coverage Δ
...ensearch/indexmanagement/IndexManagementIndices.kt 60.71% <ø> (ø)
...pensearch/indexmanagement/IndexManagementPlugin.kt 92.71% <ø> (ø)
...pensearch/indexmanagement/IndexManagementRunner.kt 55.55% <ø> (ø)
...management/common/model/dimension/DateHistogram.kt 75.60% <ø> (ø)
...ndexmanagement/common/model/dimension/Dimension.kt 85.71% <ø> (ø)
...ndexmanagement/common/model/dimension/Histogram.kt 65.07% <ø> (ø)
...ch/indexmanagement/common/model/dimension/Terms.kt 77.04% <ø> (ø)
...agement/indexstatemanagement/ISMTemplateService.kt 79.54% <ø> (ø)
...ndexstatemanagement/IndexStateManagementHistory.kt 80.14% <ø> (ø)
...nt/indexstatemanagement/ManagedIndexCoordinator.kt 71.77% <ø> (ø)
... and 259 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fe36be...0e3361e. Read the comment docs.

Signed-off-by: Clay Downs <downsrob@amazon.com>
@downsrob downsrob requested a review from annie3431 November 17, 2021 16:36
Copy link
Member

@bowenlan-amzn bowenlan-amzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@downsrob downsrob merged commit c739190 into opensearch-project:main Nov 17, 2021
@downsrob downsrob deleted the updates-copyright branch November 17, 2021 18:15
downsrob added a commit to downsrob/index-management that referenced this pull request Mar 9, 2022
* Updating license headers

Signed-off-by: Clay Downs <downsrob@amazon.com>
downsrob added a commit to downsrob/index-management that referenced this pull request Mar 9, 2022
* Updating license headers

Signed-off-by: Clay Downs <downsrob@amazon.com>
Signed-off-by: Robert Downs <downsrob@amazon.com>
downsrob added a commit that referenced this pull request Mar 9, 2022
* Updating license headers

Signed-off-by: Clay Downs <downsrob@amazon.com>
Signed-off-by: Robert Downs <downsrob@amazon.com>
wuychn pushed a commit to ochprince/index-management that referenced this pull request Mar 16, 2023
* Updating license headers

Signed-off-by: Clay Downs <downsrob@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.

None yet

4 participants