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

Logo design: Bandit #302

Merged
merged 6 commits into from May 11, 2018
Merged

Logo design: Bandit #302

merged 6 commits into from May 11, 2018

Conversation

baranpirincal
Copy link
Contributor

Hey, I uploaded all necessary logo files (editable,monochrome,full-color).

logo

@baranpirincal
Copy link
Contributor Author

@lukehinds

@lukehinds
Copy link
Member

@sigmavirus24 are we ok with the MIT license embedded (I assume not, and personally see no issue).

logo/license Outdated
@@ -0,0 +1,9 @@
MIT License
Copy link
Member

Choose a reason for hiding this comment

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

Bandit as a project has an Apache 2 license. Can you please change so we have a single license for all the source.

@baranpirincal
Copy link
Contributor Author

I changed license from mit to apache 2.0 , could you please check it?

logo/license Outdated
@@ -0,0 +1,65 @@
Apache License Version 2.0, January 2004 http://www.apache.org/licenses/
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need this file since we already have a LICENSE file that covers the entire project in the root path.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will remove it then

Copy link
Member

@ericwb ericwb left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@ericwb ericwb merged commit 3e18670 into PyCQA:master May 11, 2018
@ericwb
Copy link
Member

ericwb commented May 11, 2018

Fixes Issue #287

@ericwb ericwb mentioned this pull request May 11, 2018
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