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

build: add license headers and NOTICE file #582

Merged
merged 3 commits into from
Mar 6, 2020

Conversation

markov00
Copy link
Member

@markov00 markov00 commented Mar 6, 2020

Summary

This PRs add the required license header on every source code file of the repo.
It also correct the text for the apache license and add a NOTICE file with the bundled/based code incorporated into the library.

It also add an eslint rule to fix/lint any new file with that header.

@codecov-io
Copy link

codecov-io commented Mar 6, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@8b747af). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #582   +/-   ##
=========================================
  Coverage          ?   71.38%           
=========================================
  Files             ?      214           
  Lines             ?     6258           
  Branches          ?     1206           
=========================================
  Hits              ?     4467           
  Misses            ?     1772           
  Partials          ?       19
Impacted Files Coverage Δ
src/components/icons/assets/empty.tsx 66.66% <ø> (ø)
src/index.ts 100% <ø> (ø)
...rc/chart_types/xy_chart/renderer/dom/crosshair.tsx 79.16% <ø> (ø)
src/state/selectors/get_internal_tooltip_info.ts 75% <ø> (ø)
...hart_types/partition_chart/layout/utils/treemap.ts 10.9% <ø> (ø)
src/mocks/series/utils.ts 88.88% <ø> (ø)
.../chart_types/xy_chart/renderer/dom/highlighter.tsx 82.14% <ø> (ø)
...pes/xy_chart/state/selectors/is_tooltip_visible.ts 100% <ø> (ø)
src/specs/specs_parser.tsx 100% <ø> (ø)
src/chart_types/xy_chart/legend/legend.ts 92.1% <ø> (ø)
... and 183 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 8b747af...9ea1686. Read the comment docs.

@markov00 markov00 requested review from nickofthyme, rshen91 and monfera and removed request for nickofthyme March 6, 2020 17:48
Copy link
Contributor

@rshen91 rshen91 left a comment

Choose a reason for hiding this comment

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

LGTM!

@markov00 markov00 merged commit 5a9ec7d into elastic:master Mar 6, 2020
@markov00 markov00 deleted the 2020_03_06-oss-compliance branch March 6, 2020 18:36
@markov00
Copy link
Member Author

🎉 This PR is included in version 18.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@markov00 markov00 added the released Issue released publicly label Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Issue released publicly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants