Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added ability to seed via rust code #1061

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

DenuxPlays
Copy link
Contributor

@DenuxPlays DenuxPlays commented Dec 7, 2024

Closes: #972

TODOs

  • create generate command

@DenuxPlays DenuxPlays force-pushed the feature/seed-via-fn branch 2 times, most recently from 7c80db3 to e743474 Compare December 7, 2024 21:04
@DenuxPlays DenuxPlays force-pushed the feature/seed-via-fn branch 2 times, most recently from fa58e83 to e81c007 Compare December 7, 2024 21:24
@kaplanelad
Copy link
Contributor

Hey,
What do you think about the example that I wrote you here, #1062?

Do you still think that the seeder is necessary?

My feeling is that managing the custom seed logic can be more flexible in the before_run than the seeder cli command

@DenuxPlays
Copy link
Contributor Author

Hey, What do you think about the example that I wrote you here, #1062?

Do you still think that the seeder is necessary?

My feeling is that managing the custom seed logic can be more flexible in the before_run than the seeder cli command

Yeah could be a solution.
My point is that it gets kind of tedious to implement everything on my own.
Especially if other people want to use a good seeding system they would have to create their own system.

I also thought about customizing the auto-seeder with features like, env based seeding, seeding by tag etc.

@kaplanelad
Copy link
Contributor

If the seeder becomes available to you, what different seed functions would you write? I'm still not seeing the need for it, and perhaps I'm missing something.

@DenuxPlays
Copy link
Contributor Author

If the seeder becomes available to you, what different seed functions would you write? I'm still not seeing the need for it, and perhaps I'm missing something.

Let me create a discussion and summarize my intention maybe this will clear up things.

@DenuxPlays
Copy link
Contributor Author

@kaplanelad
Discussion: #1072

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.

Allow fixtures to execute rust code
2 participants