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

Django migrations for 5.0 #129

Closed
wants to merge 1 commit into from
Closed

Conversation

tony
Copy link
Member

@tony tony commented Apr 23, 2022

#124

  • tests: Use :memory: so it runs without erroring

  • Canonical way to generate migrations for package

    env PYTHONPATH=. DJANGO_SETTINGS_MODULE=tests.settings django-admin makemigrations robots

  • Add migration

@codecov
Copy link

codecov bot commented Apr 23, 2022

Codecov Report

Merging #129 (b2281ba) into master (7594e8c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #129   +/-   ##
=======================================
  Coverage   79.85%   79.85%           
=======================================
  Files           7        7           
  Lines         134      134           
  Branches       13       13           
=======================================
  Hits          107      107           
  Misses         20       20           
  Partials        7        7           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tony
Copy link
Member Author

tony commented Apr 23, 2022

@tony tony force-pushed the django-5.0-migration branch from b203b7a to b2281ba Compare October 15, 2022 00:25
@tony
Copy link
Member Author

tony commented Oct 16, 2022

I am going to close this, but note some newly added test instructions here: https://django-robots.readthedocs.io/en/latest/#developing

@tony tony closed this Oct 16, 2022
@tony tony deleted the django-5.0-migration branch October 16, 2022 20:35
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