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

Make it optional to generate test with model #641

Open
jacobconley opened this issue Apr 17, 2021 · 0 comments
Open

Make it optional to generate test with model #641

jacobconley opened this issue Apr 17, 2021 · 0 comments
Labels
proposal A suggestion for a change, feature, enhancement, etc s: triage Some tests need to be run to confirm the issue
Milestone

Comments

@jacobconley
Copy link

Description

Hello! The model generator is great but it always generates a test in the same directory, which isn't ideal for all use cases (not mine at least). I'd like to disable it altogether, though I imagine some other folks may just wish to put them in another directory.

I'd be happy to submit a pull request that does this if there's interest - I could just split up this call to .Box() into two right?

Steps to Reproduce the Problem

Generate a model using the Soda CLI or anything else that uses Pop's model generator

Expected Behavior

Exactly what happened, but it would be nice to be able to disable the generation of the test file that's generated alongside the requested model

@sio4 sio4 added proposal A suggestion for a change, feature, enhancement, etc s: triage Some tests need to be run to confirm the issue labels Sep 20, 2022
@sio4 sio4 added this to the Backlog milestone Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal A suggestion for a change, feature, enhancement, etc s: triage Some tests need to be run to confirm the issue
Projects
None yet
Development

No branches or pull requests

2 participants