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

Now runs with python3 (needs more testing) #537

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

PredatorFeesh
Copy link

Short description of what this resolves:

This should now actually run on python3, ALTHOUGH IT DOES REQUIRE TESTING AND UPDATES.

Changes proposed in this pull request:

  • Changed imports to work with Python3
  • Checked to make sure new imports have the same function.
  • Ran on python3 jarvis.py after installing all modules with pip3 install -3 requirements.txt

** Need some help updating and testing further, as so far only imports have been updated to work. **

@PredatorFeesh
Copy link
Author

Didn't notice someone already started this here! The code is very similar in both, but the required.txt file is better suited there and will be implemented here. Credit to skelmdev for that.

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

1 participant