Skip to content

Remove Apache license from API client generators#4197

Merged
wing328 merged 13 commits intoswagger-api:masterfrom
wing328:remove_license_2
Nov 16, 2016
Merged

Remove Apache license from API client generators#4197
wing328 merged 13 commits intoswagger-api:masterfrom
wing328:remove_license_2

Conversation

@wing328
Copy link
Copy Markdown
Contributor

@wing328 wing328 commented Nov 16, 2016

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Remove license (Apache 2) for the following API client generators:

C#, PHP, Objc, Swift, Perl Scala, TS, Go, Android, CPP

UPDATE: updated all remaining API clients (e.g. Java, Javascript, etc)

@wing328 wing328 changed the title Remove license 2 Remove Apache license from API client generators Nov 16, 2016
@wing328 wing328 added this to the v2.2.2 milestone Nov 16, 2016
@wing328 wing328 merged commit 90512e6 into swagger-api:master Nov 16, 2016
davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
* remove php apache license

* remove apache license from C#

* remove apache license in objc code

* remove license from swift 3 code

* remove apache license from perl code

* remove license from scala code

* remove license from ts, go, android, cpp, scala

* remove license from java api client

* restore clojure petstore files

* remove license from travis file

* clean up apache-related terms in php, ruby, python mustache tempaltes

* remove license from JS API cilent
@wing328 wing328 deleted the remove_license_2 branch November 30, 2017 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant