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.

Sign up for GitHub

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

Jump to bottom

fix(plaintext): include max_plaintext_per_fixture_file attribute in… #61

Merged
merged 1 commit into from
Sep 17, 2023
Merged

fix(plaintext): include max_plaintext_per_fixture_file attribute in… #61

merged 1 commit into from
Sep 17, 2023

Conversation

griff-rees
Copy link
Contributor

Include max_plaintext_per_fixture_file in PlainTextFixture.export_to_json_fixtures() call of save_fixture.

@griff-rees griff-rees added the bug Something isn't working label Sep 17, 2023
@griff-rees griff-rees added this to the v0.1.0 milestone Sep 17, 2023
@griff-rees griff-rees self-assigned this Sep 17, 2023
@griff-rees griff-rees linked an issue Sep 17, 2023 that may be closed by this pull request
Include max_elements_per_file in save_fixture call in PlaintextFixture #60
Closed
@griff-rees griff-rees merged commit 180cd00 into main Sep 17, 2023
4 checks passed
@griff-rees griff-rees deleted the 60-max_elements_per_file-for-plaintext branch September 17, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@griff-rees griff-rees

Labels
bug Something isn't working
Projects
None yet
Milestone
v0.1.0
Development

Successfully merging this pull request may close these issues.

Include max_elements_per_file in save_fixture call in PlaintextFixture
2 participants
@griff-rees @spool