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

CLN: Lint for missing whitespace around modulo operator E228,E401 #4900

Merged
merged 2 commits into from Aug 20, 2018

Conversation

jbrockmendel
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.003%) to 83.689% when pulling a3c997d on jbrockmendel:gc11 into 0251bad on statsmodels:master.

@coveralls
Copy link

coveralls commented Aug 14, 2018

Coverage Status

Coverage decreased (-0.04%) to 83.689% when pulling 87db4a4 on jbrockmendel:gc11 into 1ad915a on statsmodels:master.

@bashtage
Copy link
Member

Both of these fixes are no-brainers.

@ChadFulton ChadFulton changed the title Lint for missing whitespace around modulo operator CLN: Lint for missing whitespace around modulo operator Aug 17, 2018
@jbrockmendel jbrockmendel changed the title CLN: Lint for missing whitespace around modulo operator CLN: Lint for missing whitespace around modulo operator E228,E401 Aug 17, 2018
@jbrockmendel
Copy link
Contributor Author

@ChadFulton no merge conflict here if we can keep some momentum going

@ChadFulton
Copy link
Member

LGTM, will merge tomorrow unless concerns.

@jbrockmendel
Copy link
Contributor Author

@ChadFulton ping; no concerns have been raised

@ChadFulton
Copy link
Member

Will merge now. BTW will be busy for next two weeks. Limited response to pings.

@ChadFulton ChadFulton merged commit 3261eea into statsmodels:master Aug 20, 2018
@jbrockmendel jbrockmendel deleted the gc11 branch August 20, 2018 15:49
@josef-pkt josef-pkt added this to the 0.10 milestone Sep 16, 2018
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.

None yet

5 participants