-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add parameter for a test project location #37
Comments
It's one of those things that have come up but that we've never done. We'd happily take a PR if you feel up to the challenge! |
Heh, well, I've only looked at adding them for .net projects (vs console and xunit for dnx). I'll have a look but I've never done anything with node aside from getting the package for this so it may take a while. In the mean time, if anyone beats me to it I won't be offended :) |
Not necessarily. I think it would be fine to have this be .NET only to start with. In one of our samples, we use this logic to run tests. |
Ok, makes sense, so the PR should have logic to throw if the param is included for a non .net project? |
Yes, would be good to fail and just say it's not yet supported. |
Hi Guys,
Given that this can be used as a sort of lightweight CI service do you have any plans to add a parameter for a test project location to the template builder?
I mention it as I haven't yet used a generated script that I haven't added a test that halts deployment.
Awesome project by the way!
The text was updated successfully, but these errors were encountered: