Skip to content

Small error in order of lines in code: Chapter 10 - basic_inserts.py #14

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

Closed
aambrioso1 opened this issue Jul 28, 2020 · 2 comments
Closed

Comments

@aambrioso1
Copy link
Contributor

aambrioso1 commented Jul 28, 2020

There is a small error in basic_inserts.py. It probably went unnoticed because it only affects folks that are not using Pycharm. Here's the problem:

import pypi_org.data.db_session as db_session
from pypi_org.data.package import Package
from pypi_org.data.releases import Release

Make it run more easily outside of PyCharm

sys.path.insert(0, os.path.abspath(os.path.join(
os.path.dirname(file), "..", "..")))

The code to fix the problem with the system path needs to be place before the imports.

Thanks for the course. I am slogging along steadily.

Cheers!
Alex in Valrico, Florida

PS: Still not haven't worked out how to do pull requests.

@mikeckennedy
Copy link
Member

Hi! Thanks for this. I think it's already there, right:

Let me know if I'm missing it. :)

As for PRs, just fork the repo (make sure it's in sync: see steps 5 & 6, then git pull: https://www.dataschool.io/how-to-contribute-on-github/ ), make the change in your repo, push it to github. Visit github's page for your repo, it'll have a create PR green button, press that!

@aambrioso1
Copy link
Contributor Author

aambrioso1 commented Jul 29, 2020 via email

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

No branches or pull requests

2 participants