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

Check CI for "build -t All" for mac (linux and windows outstanding) #215

Merged
merged 15 commits into from
Sep 16, 2021

Conversation

dsyme
Copy link
Member

@dsyme dsyme commented Sep 13, 2021

This attempts to get the build working up to

dotnet fake build -t All

It is trying Windows first

The build on Ubuntu is failing with R packages not being found by the type provider. Not clear why, see #213

@dsyme dsyme changed the title Check CI for "build -t All" on Windows and Ubuntu Check CI for "build -t All" on Windows Sep 13, 2021
@AndrewIOM
Copy link
Collaborator

On my local branch I managed to get macOS working similar to the good progress by @nhirschey in #216.

I hit issues with other platforms to do with R environment configuration - there was an overlooked macos hardcoded path so this has been reconfigured to be set dynamically based on platform when starting the RProvider server.

There is an outstanding issue with running FAKE's All target on other platforms - the test project hangs forever, and requires further investigation. Nuget deployment, version tagging / release etc. is also still outstanding.

Documentation deployment is working with the macos CI setup (currently no windows or ubuntu). I'm going to merge this PR and suggest that we continue work to get all platforms working in #216 .

@AndrewIOM AndrewIOM changed the title Check CI for "build -t All" on Windows Check CI for "build -t All" for mac (linux and windows outstanding) Sep 16, 2021
@AndrewIOM AndrewIOM merged commit c85cf6f into master Sep 16, 2021
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