Summary

Add a MoveToMigrations evolution for all built-in Django apps.

Review Request #11075 — Created July 7, 2020 and submitted July 17, 2020, 5:16 p.m.

Information

Owner

chipx86

Repository

Django Evolution

Branch

master

Bugs

Depends On

Reviewers

Groups

django-evolution

People

Description

This was caused by the lack of a MoveToMigrations built-in evolution
for each of these. We set it for a few apps, just the ones that we
needed to apply our own evolutions to at first, but didn't set it for
the remaining ones.

This change adds an evolution for this for every built-in Django app
that contains a 0001_initial.py migration.

Testing Done

Evolved the database successfully. Saw the migrations all apply.

Verified that Django no longer complained about missing migrations.

Commits

Summary	ID
Add a MoveToMigrations evolution for all built-in Django apps. We had several Django apps that were still being treated as owned by Django Evolution, which meant that Django consistently warned about unapplied migrations. While mostly harmless, this could become a problem later. This was caused by the lack of a `MoveToMigrations` built-in evolution for each of these. We set it for a few apps, just the ones that we needed to apply our own evolutions to at first, but didn't set it for the remaining ones. This change adds an evolution for this for every built-in Django app that contains a `0001_initial.py` migration.	cb875f2f96811c530730747f09048e9842f549b7

Summary

Add a MoveToMigrations evolution for all built-in Django apps.

We had several Django apps that were still being treated as owned by Django Evolution, which meant that Django consistently warned about unapplied migrations. While mostly harmless, this could become a problem later. This was caused by the lack of a `MoveToMigrations` built-in evolution for each of these. We set it for a few apps, just the ones that we needed to apply our own evolutions to at first, but didn't set it for the remaining ones. This change adds an evolution for this for every built-in Django app that contains a `0001_initial.py` migration.

cb875f2f96811c530730747f09048e9842f549b7

Issues

Description	From	Last Updated
I know these are super tiny, but can we add file docstrings to the top? Same for all 4.	david	July 7, 2020, 7:46 p.m.

flake8 passed.

JSHint passed.

django_evolution/builtin_evolutions/admin_move_to_migrations.py (Diff revision 1)
The issue has been resolved. Show all issues
```
I know these are super tiny, but can we add file docstrings to the top? Same for all 4.
```

Change Summary:


Added file-level docstrings for the new evolution files.
Added missing changes to unit tests to stop using django.contrib.admin for tests involving mutation vs. evolution introspection (since it's now configured for migrations).

Commits:

	Summary	ID
	Add a MoveToMigrations evolution for all built-in Django apps. We had several Django apps that were still being treated as owned by Django Evolution, which meant that Django consistently warned about unapplied migrations. While mostly harmless, this could become a problem later. This was caused by the lack of a `MoveToMigrations` built-in evolution for each of these. We set it for a few apps, just the ones that we needed to apply our own evolutions to at first, but didn't set it for the remaining ones. This change adds an evolution for this for every built-in Django app that contains a `0001_initial.py` migration.	fe8de245f118418fe74a709f01beffba89967c7b
	Add a MoveToMigrations evolution for all built-in Django apps. We had several Django apps that were still being treated as owned by Django Evolution, which meant that Django consistently warned about unapplied migrations. While mostly harmless, this could become a problem later. This was caused by the lack of a `MoveToMigrations` built-in evolution for each of these. We set it for a few apps, just the ones that we needed to apply our own evolutions to at first, but didn't set it for the remaining ones. This change adds an evolution for this for every built-in Django app that contains a `0001_initial.py` migration.	cb875f2f96811c530730747f09048e9842f549b7

Diff:

Revision 2 (+116 -24)

Show changes

	django_evolution/builtin_evolutions/__init__.py
	django_evolution/builtin_evolutions/admin_move_to_migrations.py
	django_evolution/builtin_evolutions/contenttypes_move_to_migrations.py
	django_evolution/builtin_evolutions/flatpages_move_to_migrations.py
	django_evolution/builtin_evolutions/redirects_move_to_migrations.py
	django_evolution/builtin_evolutions/sites_move_to_migrations.py
	django_evolution/tests/test_evolution_utils.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (e94f385)