Qbeast-io / qbeast-spark

Qbeast-spark: DataSource enabling multi-dimensional indexing and efficient data sampling. Big Data, free from the unnecessary!
https://qbeast.io/qbeast-our-tech/
Apache License 2.0
202 stars 18 forks source link

Issue 290: Add correct license header to files in branch main-1.0.0 #291

Closed fpj closed 3 months ago

fpj commented 3 months ago

Description

We have corrected license headers in main, but we also have main-1.0.0 that is pending resolution and we are merging against main-1.0.0. Consequently, I'm producing this PR in the case to fix headers in main-1.0.0

WARNING: We should only commit this if it is not going to be generating conflicts when we merge main and main-1.0.0. In the case we merge main-1.0.0 into main, then we should use this PR to fix any outstanding license header issues.

Fixes #290

Type of change

This PR only touches license headers.

Checklist:

Here is the list of things you should do before submitting this pull request:

How Has This Been Tested? (Optional)

I validated that it builds correctly with: sbt assembly

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.70%. Comparing base (a1dab12) to head (5f896a2). Report is 2 commits behind head on main-1.0.0.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main-1.0.0 #291 +/- ## ============================================== - Coverage 91.08% 90.70% -0.39% ============================================== Files 95 98 +3 Lines 2569 2581 +12 Branches 316 332 +16 ============================================== + Hits 2340 2341 +1 - Misses 229 240 +11 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

osopardo1 commented 3 months ago

I think that, since we will merge main-1.0.0 to main and get rid of main-1.0.0, this PR would not be needed.

fpj commented 3 months ago

No longer necessary with the merge of PR #284.