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

Python/Django implementation #1

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

Conversation

rpkilby
Copy link

@rpkilby rpkilby commented Aug 3, 2015

Hi, I've spent a little time working on a Django version of the models and fixtures. This also includes a simple test to ensure that the models and fixtures are compatible with Django and its test utilities. To run the test suite:

$ git checkout python
$ python setup.py test

If everything looks good, what are your thoughts on releasing the package to PyPI? I'd be happy to help with/own the python releases, as well as updating the python code. It might also be worth noting that PyPI allows for multiple package owners and maintainers.

Ryan P Kilby added 10 commits August 1, 2015 17:39
- `find_packages()` does not respect `package_dir` and needs to be
  explicitly told to search a sub directory package.
- `install` and `develop` were generating the fixture in the /tmp
  directory, but the files were not copied to the target directory.
- The `test` command on python 2 uses an 'in place' install of the
  package, so the build_py phase is never executed. It's necessary for
  the test command to manually build the fixture before execution.
@rpkilby rpkilby mentioned this pull request Aug 3, 2015
3 tasks
@tkellen
Copy link

tkellen commented Aug 4, 2015

This is awesome ryan! Thank you so much. I'm going to try to review this tomorrow.

@rpkilby
Copy link
Author

rpkilby commented Sep 1, 2015

Hi - any objection to me going ahead and uploading this to pypi?

@tkellen
Copy link

tkellen commented Sep 1, 2015

None whatsoever!

@tkellen
Copy link

tkellen commented Sep 1, 2015

Also, fuck. I'm really sorry I haven't landed this already. Completely fell off the radar.

@multimeric
Copy link

Bump. This could be useful if merged.

@rpkilby
Copy link
Author

rpkilby commented Oct 8, 2019

Hi @TMiguelT fantasy-database is available on PyPI, however I'd guess it's outdated and not compatible with modern versions of Django. If you run into any issues, feel free to ping me on the issue tracker on my fork.

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