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

Fix #1879: Convert Fake.IO.FileSystem/Shell.fs to "module based" #1882

Merged
merged 3 commits into from
Apr 22, 2018

Conversation

MangelMaxime
Copy link
Contributor

@matthid I think we should untrack the .fake folder because it's specific to each computer.

@MangelMaxime
Copy link
Contributor Author

The CI is failing but I think it's coming from the code in rc_7branch and not the modifications I made.

@matthid
Copy link
Member

matthid commented Apr 22, 2018

@MangelMaxime Correct. I'm working on this problem. Seems like some breaking change after paket update (from another package)

I think we should untrack the .fake folder because it's specific to each computer.

We have to include the intellisense.fsx file for historical reasons on this repository, because FAKE 4 code needs it (bootstrapping test)

@MangelMaxime
Copy link
Contributor Author

Oh ok for the intellisense.fsx :)

Ping me on this PR if you need me to rebase it later or fix something in it.

@matthid
Copy link
Member

matthid commented Apr 22, 2018

@MangelMaxime If you want you can rebase/merge and it should be green again :)

build.fsx Outdated
@@ -828,7 +892,11 @@ Target.create "CheckReleaseSecrets" (fun _ ->
)
let executeFPM args =
printfn "%s %s" "fpm" args
#if BOOTSTRAP
Shell.exec("fpm", args=args, dir="bin")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This still seems to be missing (note that this is actually part of Fake.Core.Process)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, I just replace all Shell.PascalCase to Shell.camelCase this one was too much :)

Let's see if the CI is ok now

@matthid
Copy link
Member

matthid commented Apr 22, 2018

Ok lets see, thanks!

@matthid matthid merged commit 2b13e0d into fsprojects:rc_7 Apr 22, 2018
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.

2 participants