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

Update to dotnetstandard2.0 #45

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

Conversation

vvimjam
Copy link

@vvimjam vvimjam commented Sep 30, 2021

I've updated salesforcesharp to dotnetstandard2.0 so that it can be utilized in dotnet core & dotnet framework (4.6.1+).

Changes:

  • Updated all the nuget dependencies some of them to latest & some of them to official sources.
  • Removed HelperSharp dependency duet to its framework conflict, Dependent code is within a separate class.

I've removed pdb & xml file details from nuspec. Support for adding them in the build file is currently up in the air New project system doesn't copy PDBs from packages. It is said to be fixed in .net6.

Integrated HelpSharp code withing SalesforceSharp & removed HelperSharp dependency due to its target framework conflicts. Updated all nuget references to latest.
@vvimjam
Copy link
Author

vvimjam commented Sep 30, 2021

I'm not familiar with cake runner, but the current setup seems to be targeting .net framework which might explain why units tests are unable to discover the internal functions despite having internalvisibleto setup in the dotnetstandard project. I do not have any build issues with msbuild on my local machine.

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.

1 participant