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

CP-849 Add Apache 2.0 license, dart_dev #19

Merged
merged 1 commit into from
Aug 18, 2015
Merged

CP-849 Add Apache 2.0 license, dart_dev #19

merged 1 commit into from
Aug 18, 2015

Conversation

evanweible-wf
Copy link
Contributor

Issue

#17 Before making this repo public, we need to add the Apache 2.0 license to the repo and to all source files.

Changes

Added LICENSE and NOTICE files. Used the new copy-license task to apply this license to all source files.

Areas of Regression

  • n/a

Testing

  • n/a

Code Review

@trentgrover-wf
@maxwellpeterson-wf
@dustinlessard-wf
fyi: @jayudey-wf

@maxwellpeterson-wf
Copy link
Member

+1

@trentgrover-wf
Copy link
Contributor

+1
@jayudey-wf this isn't really ready to merge until the dependent PRs are, so don't jump on this just yet

@dustinlessard-wf
Copy link

+1, do we have guidelines on which license applies to what? Apache 2.0 for all public repos?

@evanweible-wf
Copy link
Contributor Author

@dustinlessard-wf yes, Apache 2.0 for OSS.

@evanweible-wf
Copy link
Contributor Author

@trentgrover-wf @maxwellpeterson-wf @dustinlessard-wf rebased on latest master.

@dustinlessard-wf
Copy link

+1

@maxwellpeterson-wf
Copy link
Member

+1
@jayudey-wf ready for merge

@jayudey-wf jayudey-wf changed the title Add Apache 2.0 license Add Apache 2.0 license, dart_dev Aug 18, 2015
@jayudey-wf jayudey-wf changed the title Add Apache 2.0 license, dart_dev CP-849 Add Apache 2.0 license, dart_dev Aug 18, 2015
@jayudey-wf
Copy link
Contributor

QA +1 adding license

Merging into master

jayudey-wf added a commit that referenced this pull request Aug 18, 2015
CP-849 Add Apache 2.0 license, dart_dev
@jayudey-wf jayudey-wf merged commit e0028ee into master Aug 18, 2015
@evanweible-wf evanweible-wf mentioned this pull request Aug 18, 2015
@trentgrover-wf trentgrover-wf deleted the license branch August 19, 2015 21:45
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.

5 participants