-
-
Notifications
You must be signed in to change notification settings - Fork 105
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
Implement Travis CI Build #13
Comments
It sounds like you know more about it than I do at this point :-) This would be great if you can get it working. If not, don't sweat it! |
I'll do my best, but I've never messed with it before. I'm sure I may ask tons of questions somewhere. :P -----Original Message----- It sounds like you know more about it than I do at this point :-) |
I think AppVeyor would be a good start with this since it directly supports .NET. If that's good, I'll work on Travis to get it going with Mono. |
Cool, thanks for the update. |
No problem! It seems like this should work. I'm pretty sure I can at least make sure everything compiles but I want to see if I can get all the tests run as well. |
As far as I can tell, the Travis build works nicely. So perhaps this issue can be closed? |
I was hoping to add FAKE to it so it can actually build and run the tests. Unfortunately, I haven't gotten too far with it 😞 |
@jwood803 Have you consider using AppVeyor? |
I've started to :} Maybe I should pick this back up. Would be great to learn more on FAKE. |
Cool, appveyor is a really nice tool. FAKE is something I wanted to get to learn to for a long time. Might look into it soon. |
@jwood803 I've been looking into FAKE lately, and I think I can be of assistance here. The only thing is that FAKE expects a project or solution file as far as I can tell, and we don't have one. Should we create one? |
Awesome! I might still have a branch there with my changes so far. I was attempting to compile the files directly since FAKE exposes the F# compiler. You have any thoughts on that approach?
|
I haven't looked into that yet. Do you have any sample code? |
Not complete at all but here's what I've started on - https://github.com/jwood803/xfsharp/blob/build/build.fsx
|
Thanks, I'll see if I can get something working. |
Awesome! I'll be glad to help, as well, if anything comes up.
|
I managed to get something working! You can find the code in my fake-testing branch. Here is how it works:
The build script does the following things, in order:
And that's it! All tests are then run (they all pass by the way): Now what is left is to decide are some details. For example, do we want the |
@ErikSchierboom That's awesome! Looks like you did much better than I did. :] Usually dependencies are left out and the |
@jwood803 Okay, I'll modify the script to download ll dependencies then. |
Awesome! I'll check it out over the next day or two. I also don't see why we can't do the same in the C# repository. I believe FAKE can use the C# compiler, as well. |
Well, I already searched for that but I could not find anything about Roslyn. Maybe that would require us to write a FAKE plugin? |
The CscHelper wouldn't work? Granted, I'm just speculating and haven't used it. |
Whoops! I totally missed that. I was searching for Roslyn and came up empty. This looks promising! I'll be looking at that later. |
This is fixed by PR #55. |
Heh, @ErikSchierboom I just saw this was just pushed :p https://github.com/fsharp/FAKE/blob/master/src/app/FakeLib/FscHelper.fs |
@jwood803 Yes, the |
Nice! It must have just been updated. I think I saw the last commit to it was around 24 hours ago from now. |
@jwood803 Indeed. Next up is checking to see if I can get something to work for the C# track. |
After looking at a few other projects here on GitHub I've noticed that they were able to find a way to use Travis CI with .NET projects. After a bit of searching it seems there is a way to implement this.
I can definitely take a look at implementing this with F# and C#, but I'll admit I've never messed with these kinds of builds before.
The text was updated successfully, but these errors were encountered: