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

#11 use nelmio/alice for data-faking #17

Merged
merged 16 commits into from
Apr 4, 2024
Merged

Conversation

davidhoelzel
Copy link
Member

Fake template parameters with faker and fixtures

@davidhoelzel davidhoelzel requested a review from bschultz April 2, 2024 09:20
@davidhoelzel davidhoelzel linked an issue Apr 3, 2024 that may be closed by this pull request
config/documentation.php Show resolved Hide resolved
docs/ComponentConfiguration.md Outdated Show resolved Hide resolved
src/Component/ComponentItemFactory.php Outdated Show resolved Hide resolved
src/Component/Data/Faker.php Show resolved Hide resolved
src/Component/Data/Faker.php Show resolved Hide resolved
tests/Unit/Component/ComponentItemFactoryTest.php Outdated Show resolved Hide resolved
tests/Unit/Component/Data/FakerTest.php Show resolved Hide resolved
davidhoelzel and others added 13 commits April 4, 2024 09:04
* add breakpoint config
- added forgotten documentation for breakpoints
[EA] '$item->$method()' would make more sense here (it's also faster).
Regarding best practices, the bundle routes must be prefixed with the bundle alias.
Add strict types and return types where it was missing.
- added cs-fixer rule for strict types
# Conflicts:
#	src/Component/ComponentItemFactory.php
#	src/Component/ComponentItemList.php
#	tests/Functional/Service/ComponentItemFactoryTest.php
#	tests/Unit/Component/ComponentItemFactoryTest.php
@bschultz bschultz merged commit 23adb72 into main Apr 4, 2024
2 checks passed
@bschultz bschultz deleted the issue-11/parameter-creator branch April 4, 2024 08:05
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.

Use fixtures for complex parameters
2 participants