-
Notifications
You must be signed in to change notification settings - Fork 3
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
No way to prevent building when running from Cake #2
Comments
Current plan is to add a new |
That will work.
…On Wed, Jun 7, 2017, 7:40 PM Alistair Chapman ***@***.***> wrote:
Current plan is to add a new DoNotBuild() extension method as an opposite
to ForceRebuild(), would that work @phillipsj
<https://github.com/phillipsj>?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABfthWnQ4-8fYZPJmHLVWsGhWDDp6-RDks5sBzUAgaJpZM4NyUSI>
.
|
Sweet. I'll implement that (and a quick fix for #1) tonight and hopefully release should be out shortly after :) |
@phillipsj Prerelease version |
@agc93 When I call with DoNotBuild() set I get the following
|
Oh damn sorry for dodgy patches @phillipsj I'll have a look and hopefully get these two cleaned up and working 😑 |
No need to apologise, it's an unstable build.
…On Mon, Jun 12, 2017, 12:26 PM Alistair Chapman ***@***.***> wrote:
Oh damn sorry for dodgy patches @phillipsj <https://github.com/phillipsj>
I'll have a look and hopefully get these two cleaned up and working 😑
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABfthcVNC2Z2tahJLlgD7VU4eUXdAycyks5sDWbIgaJpZM4NyUSI>
.
|
@phillipsj Sorry for the monster delay on this one, it slipped too far down the to-do list 😢 I'm finally back on a Windows machine and trying to reproduce this one now: I don't suppose the project you were using is on GitHub or anything? I think I know what it is and it's probably a standard "ClickTwice is a bit special" affair, but need to verify.. |
@phillipsj I think I've found it actually. Can you try out I think it's just a directory not existing when I look for it |
For some reason,
ForceBuild
in theClickTwiceManager
is always set totrue
, so the call toGetTargets
always uses the samePublishBehaviour
Issue reported by @phillipsj
The text was updated successfully, but these errors were encountered: