Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Allow for other python language packages. #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joaoponceleao
Copy link

This wasn't working with MagicPython (https://github.com/MagicStack/MagicPython), (or 'Python Django' for that matter). This PR takes care of that by making the checkForPythonContext more permissible.

@bh
Copy link
Owner

bh commented Oct 19, 2016

@joaoponceleao Can you please rebase your PR?

@chaffeqa
Copy link

chaffeqa commented May 9, 2017

any movement on this?

@joaoponceleao
Copy link
Author

Sorry about this. Not using atom anymore and quite busy. Will look into it this weekend.

@cclauss
Copy link

cclauss commented Oct 12, 2017

@joaoponceleao Bump on this one?

@joaoponceleao
Copy link
Author

@cclauss @bh @chaffeqa
Sorry about this everyone (trying to finish a phd and been out of atom for a long time so...).

I've rebased this by request.
I removed @lexicalunit 's PR since it did not seem to be working with magicpython on my system. It should be noted that he has his own fork of this package (although I haven't actually tried it).

@danpalmer
Copy link

Hey, ping on this, I've had to install MagicPython because the built in language-python does not work with Pyton 3.5+ syntax, but now isort isn't working.

Happy to help out get this working if I can?

@lexicalunit
Copy link
Contributor

I use MagicPython as well and atom-isort supports this.

@danpalmer
Copy link

danpalmer commented Dec 6, 2017 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants