search

HearthSim / django-reflinks

Referral links implementation for Django
MIT License
15 stars 5 forks source link

Add a manage.py file for easier development. #3

Open YPCrumble opened 5 years ago

YPCrumble commented 5 years ago

Thanks again for building this repo!

Usually when forking a django app repo I see a manage.py file - this PR simply adds one that I adapted from django-extensions.

I'm not aware of a reason not to have this but if there is one feel free to close it!

YPCrumble commented 5 years ago

I'm not sure what the failed test suite is due to - for instance I'm seeing indentation errors in admin.py but this PR doesn't touch that file.

Also, FYI, the reason I'd added the manage.py file was to make things like adding a data migration possible. If there's a way to do that without python manage.py makemigrations let me know?

jleclanche commented 5 years ago

I usually have a makemigrations.py script which looks kinda like this: https://github.com/dj-stripe/dj-stripe/blob/master/makemigrations.py

I don't usually commit it tho.

smyja commented 3 years ago

I'm not sure what the failed test suite is due to - for instance I'm seeing indentation errors in admin.py but this PR doesn't touch that file.

Also, FYI, the reason I'd added the manage.py file was to make things like adding a data migration possible. If there's a way to do that without python manage.py makemigrations let me know?

Did you get everything to work properly? If yes, can you create a repo for it?