Skip to content

Update license - #10

Merged
adityasharad merged 1 commit into
masterfrom
license
Jan 29, 2020
Merged

Update license#10
adityasharad merged 1 commit into
masterfrom
license

Conversation

@s0

@s0 s0 commented Jan 16, 2020

Copy link
Copy Markdown
Contributor

Update license following changes to canonical license.

@s0
s0 requested a review from adityasharad January 16, 2020 19:14
@hmakholm

Copy link
Copy Markdown
Contributor

This needs a corresponding change in odasa-buildutils/codeql-scripts/LICENSE.md of Semmle/code.

@s0

s0 commented Jan 16, 2020

Copy link
Copy Markdown
Contributor Author

PR opened there.

@jf205

jf205 commented Jan 17, 2020

Copy link
Copy Markdown
Contributor

@felicitymay

Copy link
Copy Markdown

Would it make sense to add a CODEOWNERS file to this repo to ensure that any changes to the license file trigger changes to related files in other repos?

@felicitymay

Copy link
Copy Markdown

Update license following changes to canonical license.

@s0 - where is the canonical license?

@s0

s0 commented Jan 28, 2020

Copy link
Copy Markdown
Contributor Author

@felicitymay

Copy link
Copy Markdown

See: https://git.semmle.com/Semmle/code/pull/36025 for the change to the license in the CodeQL CLI itself.

@felicitymay

Copy link
Copy Markdown

Given that this change has already been merged into the Security Lab and Code repositories, it makes sense to merge this too. @adityasharad - do you have any objections?

@adityasharad
adityasharad merged commit 9c56568 into master Jan 29, 2020
@henrymercer
henrymercer deleted the license branch January 29, 2020 16:28

@leuasseurfarrelds247-arch leuasseurfarrelds247-arch left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LICENSE.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants