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

[WIP] Clean up project structure #173

Closed
wants to merge 9 commits into from
Closed

Conversation

mimischi
Copy link
Collaborator

@mimischi mimischi commented Oct 20, 2017

Changelog

  • Get rid of unused files and folders.
  • Change deploy process on Dokku from buildpacks to Dockerfile.
  • Migrate from old-fasioned requirements.txt to cutting edge Pipfile (using pipenv 🍰).

To-Do

  • Make sure tests pass
  • Merge Migrate to Python 3.6 #60 before merging this PR
  • Update README.md with proposed new usage
  • Update Wiki
  • Make sure deploy to Dokku actually works

Limitations

@mimischi
Copy link
Collaborator Author

I've successfully migrated django-clock to the new proposed project structure. See related commits from mimischi/django-clock#430 and mimischi/django-clock#444.

@mimischi mimischi force-pushed the projectstructure branch 4 times, most recently from 4941265 to fa3a06a Compare December 30, 2017 10:21
* Get rid of unused files and folders.
* Change deploy process on Dokku from `buildpacks` to `Dockerfile`.
* Migrate from old-fasioned `requirements.txt` to cutting edge `Pipfile`.
* Also add new migration for Django 2
@mimischi mimischi force-pushed the projectstructure branch 2 times, most recently from 0da874d to 98bebb4 Compare December 30, 2017 10:29
@mimischi mimischi mentioned this pull request Dec 30, 2017
4 tasks
@mimischi
Copy link
Collaborator Author

Closed in favor of #225.

@mimischi mimischi closed this Dec 30, 2017
@mimischi mimischi deleted the projectstructure branch December 30, 2017 18:07
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.

1 participant