search

tattersoftware / codeigniter4-schemas

Database schema management, for CodeIgniter 4
MIT License
22 stars 10 forks source link

Fix docblock. #16

Closed sfadschm closed 3 years ago

sfadschm commented 3 years ago

Copy and pasted from myth-auth maybe? ;)

sfadschm commented 3 years ago

Just thinking: Maybe it would make sense to fix all docblocks at once? There is some wrong parameters and such.

MGatner commented 3 years ago

Definitely from Myth:Auth 😅 I wrote that collector too.

Schemas and Relations are coming up soon for my development toolkit refresh, which will include a full pass on static analysis. Certainly if you want to make a go at it first that would be great, save me some time and get it done sooner. But there will likely be some other changes to the code deeper than just the docblocks, as quite a bit of my module philosophy has changed since the last major version.

sfadschm commented 3 years ago

Sounds good. If you're going to refactor schemas anyways, I think docblocks can easily wait until then. If you need any help, just tag it as help wanted :)