Skip to content

Revamp & port to .netstandard2.1#24

Merged
dsyme merged 9 commits into
fsprojects:masterfrom
delneg:feat/netstandard2.1
May 16, 2022
Merged

Revamp & port to .netstandard2.1#24
dsyme merged 9 commits into
fsprojects:masterfrom
delneg:feat/netstandard2.1

Conversation

@delneg

@delneg delneg commented May 15, 2022

Copy link
Copy Markdown
Collaborator

Here's what's done:

  • Removed paket & fake - too much work getting them to work properly
  • Added github actions - build & tests on .netcore 3.1, .net5.0 and .net 6.0
  • Target netstandart 2.1 with the library, net7.0 with benches & netcoreapp3.1 with tests
  • Remove Travis & Appveyor - because of Github Actions
  • Remove #if MONO flags in Benches
  • Fix fsproj files - use new format

@delneg

delneg commented May 15, 2022

Copy link
Copy Markdown
Collaborator Author

cc @jackmott @marklam

@dsyme

dsyme commented May 16, 2022

Copy link
Copy Markdown
Contributor

@delneg should we look at making you a maintainer here?

@dsyme dsyme merged commit c23019a into fsprojects:master May 16, 2022
@dsyme

dsyme commented May 16, 2022

Copy link
Copy Markdown
Contributor

@jackmott let me know if there is an issue with the merge - by policy I should have waited some time before acting as backup maintainer

@marklam

marklam commented May 16, 2022

Copy link
Copy Markdown
Collaborator

@denelg I can have a go at fixing up the Fake / Paket stuff (from the Project Scaffold) if it's useful for publishing packages & docs or whatever.

@dsyme

dsyme commented May 16, 2022

Copy link
Copy Markdown
Contributor

If you need it take a look at the automated docs and package publishing here

https://github.com/fsprojects/FSharp.Control.AsyncSeq/blob/main/.github/workflows/publish.yml

@delneg

delneg commented May 16, 2022

Copy link
Copy Markdown
Collaborator Author

@delneg should we look at making you a maintainer here?

Hello, thank you for your proposal. However, given the current geopolitical situation I'm not sure that I'm a good fit for it (and I don't think that it's a good idea that someone might depend on me)

@denelg I can have a go at fixing up the Fake / Paket stuff (from the Project Scaffold) if it's useful for publishing packages & docs or whatever.

Anyway, regarding Paket and Fake - in my few years of experience they just make thinks harder to work with, unless in specific cases (when you can install nuget from git repo for example with Paket)
But overall I've found that they just bring not always necessary complexity to the package, and for one like this I think everything works just fine with dotnet build on all platforms (I've currently tested windows and linux both on x86_64, but can also test on macos x86_64 / arm64 if needed)

If you need it take a look at the automated docs and package publishing here
https://github.com/fsprojects/FSharp.Control.AsyncSeq/blob/main/.github/workflows/publish.yml

If you decide to fix them it's could be useful, but personally I'd prefer to use the aforementioned github actions used for AsyncSeq for the tasks of publishing packages & docs

@marklam

marklam commented May 16, 2022

Copy link
Copy Markdown
Collaborator

I've made a PR for the github actions approach, and a couple of tweaks to the project file to populate the package info & set a version (I set it to beta just because the previous published release was beta)

(sorry for mis-spelling your id in the previous comment, @delneg)

@delneg

delneg commented May 16, 2022

Copy link
Copy Markdown
Collaborator Author

I've made a PR for the github actions approach, and a couple of tweaks to the project file to populate the package info & set a version (I set it to beta just because the previous published release was beta)

(sorry for mis-spelling your id in the previous comment, @delneg)

Thanks ! Looks great , probably only need to swap dotnet 5 for dotnet 6 (as dotnet 6 is LTS and dotnet 5 is EOL)
(no problem regarding misspelling it's not an issue at all)

@marklam

marklam commented May 16, 2022

Copy link
Copy Markdown
Collaborator

Thanks, added that change to #26

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