-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Ability to override config #23
Comments
I'll give this some thoughts, thanks! In the meantime you could set a different |
Also if possible please do let me know if that's not sufficient in your use case and why. |
My use case was more for running tests in the same process, for when I'm developing. I must admit that I'm still not 100% clued in to how testing with spec/ghostwheel should be done, so I might have a suboptimal workflow or just missing something. Basically, I use kaocha for running tests, and then when developing tests I just send the test to the repl using I wanted to have a test that does the ghostwheel checks by just calling ghostwheel.core/check inside the test. But to do that, I needed to override the config, as the project config is set up not to do extensive checks. I thought it would be possible to some type of dynamic binding to override the config, but looking at the code it doesn't seem possible, |
This sounds like it should be a thing, coming with the next release. :) |
As for the testing workflow, this part of the documentation talks about it a bit: |
👋 Hi. I'm posting this same comment to all issue threads to just give a quick heads up that the project, despite rumours and some evidence to the contrary, is not dead. It was hibernating for a little while and now nearing the long-awaited next release, which will fix some long-standing issues (and introduce some breaking changes to the config). I'll be reviewing all open issues and PRs over the next couple of weeks, so stay tuned and thanks for the patience. See also this Slack thread |
It would be useful to override the configuration when running checks, as I would like to run checks as part of a test suite. Maybe some kind of dynamic var that is nil by default but gets merged in inside the
merge-config
file?The text was updated successfully, but these errors were encountered: