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

Feature: Add more standard targets #281

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MISTikus
Copy link

@MISTikus MISTikus commented Apr 4, 2023

Added netstandard2.0 and netstandard2.1 targets to avoid incorrect dependencies for projects targenting this frameworks.

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration .github/workflows/codeql-analysis.yml:build. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@snakefoot
Copy link
Contributor

@MISTikus Maybe revert the last commit, and keep using net5.0 (Since build-server is now failing). Maybe remove netstandard2.1 since netstandard2.0 is good enough.

@dustinchilson Any chance of having this merged, since it will solve a lot of dependency issues for people using NET6 (or newer)

@MISTikus
Copy link
Author

MISTikus commented Oct 30, 2023

@MISTikus Maybe revert the last commit, and keep using net5.0 (Since build-server is now failing). Maybe remove netstandard2.1 since netstandard2.0 is good enough.

@snakefoot, done. I removed last commit

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