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
Switch GATK to the Apache 2.0 license #7079
Conversation
@lbergelson please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments and updates. I also went through and committed other updates to the license that I found in other files in this repo.
Co-authored-by: Jonn Smith <jonn-smith@users.noreply.github.com>
Co-authored-by: Jonn Smith <jonn-smith@users.noreply.github.com>
7f53928
to
0b65f12
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just did my own pass for any remaining License language that I could find that could cause problems with the new language. Besides bits that get pulled in by our build/dependencies I see a few python files with old license agreements CreateCosmicFusionGene.tsv
, CreateCosmicGene.tsv
, and GenericTsvReader.py
. We should probably ask about whether those can be done away with. They look like old oncotator code perhaps?
@@ -8,15 +8,6 @@ | |||
import org.broadinstitute.hellbender.tools.walkers.varianteval.util.Analysis; | |||
import org.broadinstitute.hellbender.tools.walkers.varianteval.util.DataPoint; | |||
|
|||
/** | |||
* The Broad Institute |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just did a check and these 3 files seem to be the last vestiges of old license code in the headers of any of our source files.
No description provided.