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

CLAs #64

Open
hyandell opened this issue Feb 17, 2021 · 1 comment
Open

CLAs #64

hyandell opened this issue Feb 17, 2021 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@hyandell
Copy link
Contributor

I miss not having the CLA side of things in the diff tool.

How can we get to a CLA-askalono? Happy to do some work if you can mentor :)

@jpeddicord
Copy link
Owner

Yep! I'm open to it. There's a couple approaches off the top of my head:

  1. If this is for your own internal use, we can hack on something to import those into a Store (and compile it into the binary) and keep that in the examples directory. This would basically be abusing the LicenseEntry struct to store a CLA instead -- they're close enough that it's not too hacky.

  2. If you want to upstream this with CLA support baked in by default, there are a couple more tasks:
    a. Some kind of marker or abstraction to indicate that a stored LicenseEntry is actually a CLA. I'd be OK with an enum that indicates the type of a license (which for now would be something like SoftwareLicense or ContributorLicenseAgreement, etc).
    b. Perhaps some fields on the API that distinguish whether you want to search licenses, CLAs, or both/all types. But that is a more invasive change that probably doesn't need to happen yet (I doubt askalono will get confused differentiating licenses vs CLAs).
    c. Some kind of authoritative source of CLA texts, ala SPDX. Doesn't have to be the same format, just a location to refer to from here. I don't want to store that directly in this repository, and would prefer to not maintain a textual repository of that data elsewhere (but might cave on that).
    - The CLA data set could be omitted if that's something you'd want to insert at build time on your systems.

@jpeddicord jpeddicord added enhancement New feature or request good first issue Good for newcomers labels Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants