Skip to content

Fix #728, pass rope project to ImportTools init #802

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

Merged
merged 1 commit into from
Feb 6, 2018

Conversation

boatx
Copy link

@boatx boatx commented Oct 28, 2017

Fix for #728 . As noted by @pieterdd and @soulomoon calling autoimport results in AttributeError. Rope ImportTools expect to get Project object as input argument to init function not PyCore object.

@fmv1992 fmv1992 force-pushed the develop branch 4 times, most recently from 3f4fc36 to 60c638d Compare November 22, 2017 02:23
@diraol
Copy link
Contributor

diraol commented Feb 6, 2018

Hi @boatx Can you rebase you branch against the develop branch again and then squash the merge commits? to keep the history a little bit cleaner.

Despite that, It seems to me that this PR can be merged. It may also impact on #845, from @kierun .

@boatx
Copy link
Author

boatx commented Feb 6, 2018

Hi @diraol I've rebased my branch against develop.

@diraol
Copy link
Contributor

diraol commented Feb 6, 2018

Hi @fmv1992 I think this is ready to be merged! =)

@fmv1992 fmv1992 merged commit b8c9ece into python-mode:develop Feb 6, 2018
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.

3 participants