disable migrations config

[gary-alway]

What does it do?

• Disable migrations given the runMigrations settings key
• Defaults to true if not present

Why is it needed?

Model migrations mean that old migrations might not be compatible with the current schema which makes local development fail for fresh dbs.

How to test it?

Passing runMigrations = false will prevent any migration scripts from running, by default migrations remain enabled.

Related issue(s)/PR(s)

strapi/documentation#1353

[strapi-cla]

All committers have signed the CLA.

[gary-alway]

@alexandrebodin any news on this PR please ?

[alexandrebodin]

I wasn't added as reviewer so didn't see this PR.

[codecov]

Codecov Report

Base: 60.49% // Head: 60.49% // No change to project coverage 👍

Coverage data is based on head (e912c0a) compared to base (3446db4).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #14593   +/-   ##
=======================================
  Coverage   60.49%   60.49%           
=======================================
  Files        1352     1352           
  Lines       33179    33179           
  Branches     6339     6340    +1     
=======================================
  Hits        20070    20070           
+ Misses      11274    11273    -1     
- Partials     1835     1836    +1     

Flag	Coverage Δ
back	50.33% <0.00%> (ø)
front	65.05% <ø> (ø)
unit_back	50.33% <0.00%> (ø)
unit_front	65.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
packages/core/database/lib/index.js	29.26% <ø> (ø)
packages/core/database/lib/migrations/index.js	30.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[alexandrebodin]

Ok so general thought about the config name. we like to make the naming more declarative instead of enableX disableX, what do you think of runMigrations: true|false ?

[alexandrebodin]

Hey I'm seeing the CLA-sign not very happy probably because you used 2 different users to commit can you check that please :)

[pwizla]

Hi! I'm Pierre, a technical writer at Strapi. Thank you for the contribution, @gary-alway!
This PR would require a small documentation update, thus creating a PR for Strapi's documentation repo. If I understand correctly, we'd need to add a line to the table in this section to describe the runMigration parameter.

Would you like to do it yourself, or would you rather have me handle it? 🙂

[gary-alway]

thanks @pwizla, I will have a go at updating the docs

[pwizla]

Hi! The related documentation PR is approved. Just waiting for this PR to be merged in strapi/strapi before merging the docs one :)

[gary-alway]

thanks @pwizla, @alexandrebodin 🚀
keep up the great work on Strapi, its a pleasure to work with