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

Add the Apache 2.0 license #67

Merged
merged 1 commit into from
Oct 1, 2019
Merged

Add the Apache 2.0 license #67

merged 1 commit into from
Oct 1, 2019

Conversation

P4rk
Copy link
Contributor

@P4rk P4rk commented Sep 27, 2019

For people to be able to use this SDK all the distributed source files need to be prefixed with a license. I have also added a LICENSE file to the root of the project.

The LICENSE file is copied in from http://www.apache.org/licenses/LICENSE-2.0.txt
The prefix of each of the files is implemented as directed in the appendix of the apache license. Which can be found at the bottom of the LICENSE file. APPENDIX: How to apply the Apache License to your work.

It would be great to get this merged as I cannot use this SDK until the license on the source files are implemented.

Add the Apache 2.0 license to the root of the project and prefix all distributed source with
the the Apache 2.0 license
Copy link
Contributor Author

@P4rk P4rk left a comment

Choose a reason for hiding this comment

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

I don't believe that the PyDoc changes that I have made should cause the Travis build to fail.

@contygm
Copy link
Contributor

contygm commented Oct 1, 2019

@P4rk Your PyDoc changes aren't causing the Travis build to fail. Branches that aren't created from the main repo (aka PRs from forks) don't have access to the environment variables.

We will run tests, merge and release a patch shortly

@P4rk
Copy link
Contributor Author

P4rk commented Oct 1, 2019

That's great @contygm. Thanks.

@Avalara-ChrisWalker Avalara-ChrisWalker merged commit acef0f9 into avadev:master Oct 1, 2019
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.

None yet

3 participants