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

ADHOC: Add license #6

Merged
merged 2 commits into from
Jun 15, 2021
Merged

ADHOC: Add license #6

merged 2 commits into from
Jun 15, 2021

Conversation

sfong-roblox
Copy link
Contributor

@sfong-roblox sfong-roblox commented Jun 15, 2021

What problem are we solving?

Doesn't have license in files and in README

How are we solving the problem?

Adding the license per legal's advice

After everything is approved, we typically release under MIT.

Approval checklist

By approving this PR, you confirm with me that the following are true:

  • We have all rights to release it, i.e., we wrote it all so we own it all.
  • The release has been reviewed and approved at the VP level. We want to be sure that the business justification for the release has been vetted. (This will be the last of the checklist)
  • There are no potential Roblox patents or inventions included in the release and no patent claims threatened against the released code.
  • There are no trade secrets or other sensitive information included in the release.
  • There is no strong encryption in functionality associated with the release.

Checks

Please check these off before promoting the pull request to non-draft status.

  • All CI checks are green.
  • I have reviewed the proposed changes myself.

@sfong-roblox sfong-roblox marked this pull request as ready for review June 15, 2021 15:46
@sfong-roblox sfong-roblox requested a review from a team as a code owner June 15, 2021 15:46
@sfong-roblox sfong-roblox merged commit b80088c into main Jun 15, 2021
@sfong-roblox sfong-roblox deleted the sfong-roblox/ADHOC-add-license branch June 15, 2021 20:15
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.

2 participants