Skip to content

Conversation

@markstory
Copy link
Member

Add documentation for new parameter less mode for migrations seed

# or
$ bin/cake migrations seed --seed UserSeed
# One of
bin/cake migrations seed --seed User
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should IMO remove the option part for the major, it become superfluous and will only bloat docs and maybe confuse people in the long run.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, afaik I did that already ( https://github.com/cakephp/migrations/pull/935/files )
I must have been forgetting the docs update.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we should be breaking compatibility just for fun.

Copy link
Member

@dereuromark dereuromark Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats not for fun
The current way is bonkers and completely wrong
The major allows this cleanup - as with all other commands the subset (one specific) of all is an argument never an option. E.g. bake.

Seed Options
------------
Additional option
Copy link
Member

@dereuromark dereuromark Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Additional option
Additional options

maybe

@dereuromark
Copy link
Member

@markstory Maybe hold of with this PR until #939 is done
I am fully fixing the docs there, so this would create merge conflicts.

@dereuromark dereuromark marked this pull request as draft November 4, 2025 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants