FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update : Command for python 3 added by Shivang-Bhandari · Pull Request #73 · pythonindia/inpycon2017 · GitHub

Repository navigation

Update : Command for python 3 added - #73

Open
Shivang-Bhandari wants to merge 3 commits into
pythonindia:masterfrom
Shivang-Bhandari:master
Open

Shivang-Bhandari wants to merge 3 commits into
pythonindia:masterfrom
Shivang-Bhandari:master

Conversation

Copy link
Copy Markdown

Updated the command to run the project locally using Python 3

Copy link
Copy Markdown
Author

@akshayaurora would be great if you could take a look.

realslimshanky commented Jul 21, 2017 •
edited
Loading

Copy link
Copy Markdown
Member

@Shivang-Bhandari please specify the python version you are using.
I deployed your fix and running staticjinja on Python 3.5.2 (other versions might also) is throwing error while importing easywatch which is a currently an open issue with a temporary solution.
It's better to go with the solution which works i.e. Python 2 in this case.

Shivang-Bhandari commented Jul 22, 2017 •
edited
Loading

Copy link
Copy Markdown
Author

@realslimshanky Python 3.6.0
If you feel we should stick to Python 2 then it's not a problem.

Copy link
Copy Markdown
Member

@Shivang-Bhandari If you have tested it on 3.6.0 specifically do mention in the README about it. Because other versions might fail to run the script.

Copy link
Copy Markdown
Author

@realslimshanky sure

Comment thread README.md

For Python 3.6.0 :
```
python build.py & python -m http.server && fg

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This should work for all versions > 3.3

Its better to mention that, 3.6 was released recently, but this should work with relatively newer python 3 versions

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@SanketDG in the above comments @realslimshanky said it was not working for him for version 3.5.2, it would be great if you could update me and I can make the necessary changes. :)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Sorry for the late response, but the above code should work for any versions > 3.3 since there have been no breaking changes after that.

So you can go ahead and change that

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@SanketDG Done

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Would be great if you could merge it
:)

Copy link
Copy Markdown
Member

@SanketDG would you like to have another look after the update?

Copy link
Copy Markdown

@realslimshanky LGTM

This branch has not been deployed

No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL