Skip to content

Add opendex to the Pylint path - #7114

Closed
andy-armstrong wants to merge 2 commits into
masterfrom
andya/pylint-for-openedx
Closed

Add opendex to the Pylint path#7114
andy-armstrong wants to merge 2 commits into
masterfrom
andya/pylint-for-openedx

Conversation

@andy-armstrong

Copy link
Copy Markdown
Contributor

No description provided.

@andy-armstrong
andy-armstrong force-pushed the andya/pylint-for-openedx branch from 8a8cc41 to b628a37 Compare February 26, 2015 00:22
@andy-armstrong
andy-armstrong force-pushed the andya/pylint-for-openedx branch from b5d0cbe to 60eba5d Compare February 26, 2015 15:15
@sarina

sarina commented Feb 27, 2015

Copy link
Copy Markdown
Contributor

I think this is beind done in #6998 - can you take a look and see if I'm right/wrong about that?

@andy-armstrong

Copy link
Copy Markdown
Contributor Author

Thanks, @sarina. Yep, #6998 looks more comprehensive. I think some of the pep8 fixes I made may now be needed on that branch before it can merge. Rebasing should expose them, I would think.

@andy-armstrong

Copy link
Copy Markdown
Contributor Author

Closing in favor of #6998.

@andy-armstrong
andy-armstrong deleted the andya/pylint-for-openedx branch February 27, 2015 17:42
@sarina sarina mentioned this pull request Mar 13, 2015
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.

2 participants