Skip to content
This repository was archived by the owner on Jun 27, 2020. It is now read-only.

Fix flake8 Travis issues#67

Merged
rohithasrk merged 1 commit intoopenwisp:masterfrom
AYZhu:master
Dec 2, 2017
Merged

Fix flake8 Travis issues#67
rohithasrk merged 1 commit intoopenwisp:masterfrom
AYZhu:master

Conversation

@AYZhu
Copy link
Copy Markdown
Contributor

@AYZhu AYZhu commented Nov 30, 2017

No description provided.

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling e2ad240 on AZtheAsian:master into ** on netjson:master**.

1 similar comment
@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling e2ad240 on AZtheAsian:master into ** on netjson:master**.

Copy link
Copy Markdown
Contributor

@rohithasrk rohithasrk left a comment

Choose a reason for hiding this comment

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

Nice work @AZtheAsian. Change the commit message to something like [QA] Fixed flake8 errors in Travis build and you're ready to go. Otherwise LGTM 😄

@nemesifier
Copy link
Copy Markdown
Member

@AZtheAsian great work 👍 , please read about our commit message style

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 1, 2017

Coverage Status

Changes Unknown when pulling ed682a0 on AZtheAsian:master into ** on netjson:master**.

@rohithasrk rohithasrk merged commit 020617d into openwisp:master Dec 2, 2017
@rohithasrk
Copy link
Copy Markdown
Contributor

Merged @AZtheAsian. Thanks ;)

@nemesifier
Copy link
Copy Markdown
Member

nemesifier commented Dec 2, 2017

👍 nice work @AZtheAsian

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants