Skip to content

Conversation

@pinzart90
Copy link
Contributor

@pinzart90 pinzart90 commented Jan 6, 2023

This PR does:
Added remaining projects to the dotnet6 build
Added new Pythonnet assemblies targeting netstandard2.0 (will remove when pythonnet PR is merged)
Fixed a bug with the way we search for ASM files on linux vs windows
Some small changes from @saintentropy

@@ -0,0 +1 @@
These assemblies are built from https://git.autodesk.com/Dynamo/pythonnet targeting netstandard2.0 No newline at end of file
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added new assemblies for netstd20 only because I could not get tests to run on the pythonnet targeting netstd20
Once we get pythonnet CICD to run properly, we can just keep the new netstd20 assmeblies (since they are compatible with net48 and net6.0)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did the test run properly for you locally? The embedded test part of CICD for PythoNet is unstable (when running in docker) and has been for a while.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, we have more changes coming into python net soon so we should resolve this sooner than later.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could not get the tests to run locally.
Something with the nunit version is causing issues.

Copy link
Contributor

@sm6srw sm6srw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pinzart90 pinzart90 merged commit d2b353a into master Jan 30, 2023
@pinzart90 pinzart90 deleted the cross_platform_part2 branch January 30, 2023 12:52
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.

5 participants