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

Issue296 #376

Merged
merged 17 commits into from
Mar 28, 2014
Merged

Issue296 #376

merged 17 commits into from
Mar 28, 2014

Conversation

MorganPersson
Copy link
Contributor

Code changed to use NancyFx.

@forki
Copy link
Member

forki commented Mar 28, 2014

wow. cool stuff.

/cc @colinbull

@MorganPersson
Copy link
Contributor Author

The build fails, but I don't know whats the error means.

"/Users/travis/build/fsharp/FAKE/build.fsx(3,1): error FS0078: Unable to find the file 'tools/SourceLink.Fake/tools/SourceLink.fsx' in any of
/Users/travis/build/fsharp/FAKE/tools/FAKE/tools/
/Users/travis/build/fsharp/FAKE

@forki
Copy link
Member

forki commented Mar 28, 2014

that's not related to your PR - see #345

@colinbull
Copy link
Contributor

Nice... I'm looking forward to having a play with this.

forki added a commit that referenced this pull request Mar 28, 2014
@forki forki merged commit f803bb3 into fsprojects:develop Mar 28, 2014
@forki
Copy link
Member

forki commented Mar 28, 2014

since there are so many changes I merged this and we should work/fix from here.

@forki
Copy link
Member

forki commented Mar 28, 2014

image

@forki
Copy link
Member

forki commented Mar 28, 2014

but stange thing is: if I call build.cmd these don't get reported.

@MorganPersson
Copy link
Contributor Author

Thats because build.cmd dont run these tests, I wrote them in F# with
xunit. I cant stand mspec in C#, its so damn ugly.
I can put that into build.cmd, but I have to do some "real" work first...

On Fri, Mar 28, 2014 at 11:39 AM, Steffen Forkmann <notifications@github.com

wrote:

but stange thing is: if I call build.cmd these don't get reported.

Reply to this email directly or view it on GitHubhttps://github.com//pull/376#issuecomment-38906372
.

@forki
Copy link
Member

forki commented Mar 28, 2014

yes no hurry.

@forki
Copy link
Member

forki commented Mar 30, 2014

@forki
Copy link
Member

forki commented Mar 30, 2014

image

@MorganPersson
Copy link
Contributor Author

I think you have already figured it out?
3c4c4b1

On Sun, Mar 30, 2014 at 11:13 AM, Steffen Forkmann <notifications@github.com

wrote:

[image: image]https://cloud.githubusercontent.com/assets/57396/2560899/982cf6fc-b7eb-11e3-87c5-f0f02d3a8d8c.png

Reply to this email directly or view it on GitHubhttps://github.com//pull/376#issuecomment-39021058
.

@forki
Copy link
Member

forki commented Mar 30, 2014

/home/monoagent/.build-agent/work/689cb4747b596ebb/src/deploy.web/Fake.Deploy.Web/Extensions.fs(15,11): The type 'ExtensionAttribute' is not defined

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants