[qa] Simplify migration checks

[nemesifier]

We currently have two migration checks:

• one checks whether there are pending migrations to create (./manage.py makemigrations --dry-run)
• one checks whether the names of the migration files have auto in them

The problem is that we have to specify both --migration-module and --migration-path.
I think we can avoid doing this and have only one command line flag.

I would get rid of --migration-path, keeping only --migration-module, although I would rename this flag to --migration-modules while keeping backward compatibility with the old flag.

--migration-path should be ignored (printing a deprecation warning).

We will need to upgrade the modules using this: https://github.com/search?q=org%3Aopenwisp+%27--migration-path%27&type=Code

[nemesifier]

@atb00ker @ppabcd do you see any issues in this?

[atb00ker]

I don't see an issues, all migrations are in the migrations/ folder and the script can look for that folder name to find the migrations! 😄

[AditiSah05]

Thanks for the detailed explanation! I’d like to work on this issue.

Here’s my understanding of the proposed changes:

Combine the two existing migration checks so that only --migration-module (renamed to --migration-modules) is needed

Keep --migration-module for backward compatibility

Mark --migration-path as deprecated and ignore it, while showing a deprecation warning

Update all affected OpenWISP modules that still use --migration-path

If this approach looks correct, I can start working on the implementation and submit a PR.

Please let me know if there are any additional considerations I should keep in mind.