Skip to content

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.

6 participants