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

PEP-8 conformance improvements #3052

Merged
merged 1 commit into from
Nov 16, 2013
Merged

PEP-8 conformance improvements #3052

merged 1 commit into from
Nov 16, 2013

Conversation

timleslie
Copy link
Contributor

Updates to make master branch pep8 clean once again.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 24f462856766f26786574e3a2dd543bfd1eba56a on timleslie:master into b73ec23 on scipy:master.

@rgommers
Copy link
Member

Hi Tim, it's only the last commit that needs to go in right? Something always goes wrong if you merge master into your feature branch - the clean way to do this would be to just update your master to be in sync with scipy master, create a new branch from there, and then make your changes.

@timleslie
Copy link
Contributor Author

You're right, it is only the latest commit that needs to go in. Let me
clean up my branches and I'll resubmit the PR more cleanly.

On Tue, Nov 12, 2013 at 8:01 AM, Ralf Gommers notifications@github.comwrote:

Hi Tim, it's only the last commit that needs to go in right? Something
always goes wrong if you merge master into your feature branch - the clean
way to do this would be to just update your master to be in sync with scipy
master, create a new branch from there, and then make your changes.


Reply to this email directly or view it on GitHubhttps://github.com//pull/3052#issuecomment-28238298
.

@timleslie
Copy link
Contributor Author

Hi Ralf,

I've fixed up this PR to be a single commit directly on top of the current scipy/master. Should be good to go.

Cheers,

Tim

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 6a6c66d on timleslie:master into f9386ac on scipy:master.

rgommers added a commit that referenced this pull request Nov 16, 2013
PEP-8 conformance improvements
@rgommers rgommers merged commit 11d8bee into scipy:master Nov 16, 2013
@rgommers
Copy link
Member

Thanks Tim, looks good.

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