Skip to content

Conversation

@aledista
Copy link
Contributor

No description provided.

@kastiglione
Copy link
Contributor

Thanks for the pull request! Was this done using a tool? Could you update the pull request to leave out all the indentation, whitespace, and line wrapping changes? It would be great to remove unused vars, but I'd rather avoid reformatting the entire code base.

@aledista
Copy link
Contributor Author

I changed my commit. I left out style conventions and removed unused imports, vars and fixed a couple of minor issues. I generally use PyCharm to identify issues like that.

Copy link
Contributor

Choose a reason for hiding this comment

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

This block shouldn't be removed. recursionName is used below.

@aledista
Copy link
Contributor Author

Readded 👍

kastiglione added a commit that referenced this pull request May 13, 2015
pep8; removed unused vars
@kastiglione kastiglione merged commit 22d74dd into facebook:master May 13, 2015
@kastiglione
Copy link
Contributor

Thanks again for the pull request!

@aledista
Copy link
Contributor Author

You're welcome!

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.

3 participants