google / flatbuffers

FlatBuffers: Memory Efficient Serialization Library
https://flatbuffers.dev/
Apache License 2.0
22.56k stars 3.19k forks source link

Fix license details #8210

Closed nfriedly closed 4 months ago

nfriedly commented 6 months ago

We updated to the latest release of the flatbuffers javascript library from npm, and Arnica threw up a warning that flatbuffers was using a non-standard license. However you're actually using a completely standard Apache 2.0 license, it was just mislabeled. (I diffed it against https://www.apache.org/licenses/LICENSE-2.0.txt to be sure and noticed that it didn't even have the copyright info filled in!)

So, this PR fixes two things:

  1. package.json now uses the correct SPDX license identifier.
  2. The copyright year and owner in the LICENSE file is now filled in

Feel free to edit that as appropriate. I can also split this into two PRs if preferred.

google-cla[bot] commented 6 months ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

nfriedly commented 6 months ago

I am now a member of FullStory's corporate CLA group, but I'm not sure how to refresh the check here to recognize that. Nevermind, found the link at the bottom of of the cla check failure message.

p0fi commented 4 months ago

Could this pull request be merged? We are experiencing a similar issue with our license scan, which is complaining about the license not being allowed, even though it is actually completely allowed.

dbaileychess commented 4 months ago

Fixed: #8253