cakephp / phinx

PHP Database Migrations for Everyone
https://phinx.org
MIT License
4.45k stars 895 forks source link

Update docs for 0.16.x #2279

Closed ajibarra closed 2 months ago

ajibarra commented 3 months ago

Fixes #2271

MasterOdin commented 2 months ago

Thanks!

MichaelRushton commented 2 months ago

@MasterOdin @ajibarra

This doesn't actually update the documentation, it only changes the displayed version number.

As an example, this code from the documentation throws a fatal error because getQueryBuilder now requires a string $type argument.

$builder = $this->getQueryBuilder();

There are also now getSelectBuilder, getInsertBuilder, getUpdateBuilder, and getDeleteBuilder methods that are undocumented.

MasterOdin commented 2 months ago

The docs for that hasn't been updated, and can see that in git: https://github.com/cakephp/phinx/blob/615485032e67d9500cbfa2e53acdb0449e26f191/docs/en/migrations.rst?plain=1#L1646-L1676

The docs are ideally rolling in that we only publish for the latest 0.x release, but things get missed. I'll make an issue to at least keep track of this particular issue, but please make issues for any other things you notice are out of date.