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

Consolidating supported TargetFrameworks to netstandard2.0 #1

Closed
3 tasks
Arlodotexe opened this issue Jun 23, 2022 · 1 comment · Fixed by #4
Closed
3 tasks

Consolidating supported TargetFrameworks to netstandard2.0 #1

Arlodotexe opened this issue Jun 23, 2022 · 1 comment · Fixed by #4
Assignees
Labels
maintainer priority The repo maintainers are prioritizing this

Comments

@Arlodotexe
Copy link
Collaborator

Arlodotexe commented Jun 23, 2022

Background

We're forking and maintaining the amazing work done by Richard Schneider, who appears to have permanently logged off since November 2019.

To support IPFS in the .NET ecosystem, I'll be maintaining this repo and the net-ipfs-core repo, approving PRs, and generally helping the community pick up where Richard left off.

First steps

Before we can have PRs rolling in, one of the first things we need to do is consolidating our TargetFramework to netstandard2.0.

There's a few reasons for this:

  • net45 has since reach end of life
  • netstandard1.4 is starting to show its age (also just not recommended anymore)
  • As pointed out in #70, the cs-multibase library recommended by IPFS only supports netstandard1.6. This is a complex thing to create and is becoming an increasingly important part of the spec, so we're bringing this dependency along.
  • All .NET implementations (core, framework) that support anything under 2.0 are EOL by now.

Checklist

  • All TargetFrameworks have been removed, except for netstandard2.0
  • All compilation conditionals (#if NET45, etc) have been removed.
  • Tests have been consolidated to a single .NET 6 project, referencing a single NS2.0 project
@ATikadze
Copy link
Contributor

👋

@Arlodotexe Arlodotexe assigned ATikadze and unassigned Arlodotexe Jun 24, 2022
@Arlodotexe Arlodotexe added the maintainer priority The repo maintainers are prioritizing this label Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer priority The repo maintainers are prioritizing this
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants