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

Modernise build process and tooling #159

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

AndrewIOM
Copy link

@AndrewIOM AndrewIOM commented Apr 9, 2024

Motiviation

This PR includes a modernisation of the workflow for building and testing R.NET, including cross-platform CI and all projects on the latest SDK format. The library still targets netstandard2.0, so there should be no change to .NET framework compatibility. It should hpoefully make R.NET easier to develop and contribute bug fixes going forward.

As a maintainer on the RProvider project, I would like for R.NET to become a more active project again. We have various outstanding bugs and features that are dependent on changes in R.NET. Following this PR, I can send in further PRs for things we need for RProvider, or if current maintainers are low on time commitment I can potentially offer some time to help maintain the repo. This could offer some direction for re: #158. Thanks!

Changes

  • FAKE build migrated to v6 (.NET 6 compatible version).
  • Use FAKE to specify nuget package properties rather than nuspecs or proj files.
  • All project files moved to .NET SDK format (if not already).
  • Use paket for dependency management (removed all other package references).
  • Set up Github Actions for CI - builds and tests (note: commented out in the Actions file is a part to automatically release nuget packages when a version is tagged on github. This is what we use in RProvider).
  • Reorganised projects into src and test folders and removed redundant files related to .NET framework / mono building and other old build processes (e.g. nuget.exe process).
  • Update readme to reflect the latest changes to the build and test process.
  • Add status icons to readme for github actions and nuget.

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.

None yet

1 participant