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

Removed unpack behaviour and use "runtimes" feature of nuget packages #249

Merged
merged 33 commits into from
Jul 16, 2024

Conversation

timyhac
Copy link
Collaborator

@timyhac timyhac commented Apr 12, 2022

No description provided.

@jkoplo
Copy link
Member

jkoplo commented Apr 12, 2022

Ooh, does this work? Just read up on it.

@timyhac
Copy link
Collaborator Author

timyhac commented Apr 12, 2022

@jkoplo - I'm not sure..
It's been on "the list" since before this repo was public, but I'm not sure how it works, and whether it supports as many scenarios as the Extractor method.
I figured that I'd get it to a level of completion that at least be tested.

timyhac added 2 commits June 30, 2024 17:24
* Split up types into NativeImport, .Common, .NetCore and .NetFramework packages
* Update all Examples to work.
* Won't work online because new packages are required.
timyhac added 19 commits July 3, 2024 20:22
…elevant when using

Nuget/MSBuild runtimes feature
Then adjust the Build script to only use artifacts folder for NativeImport testing
* A project each combination of NetCore/NetFramework and Direct/Transitive dependencies.
Signed-off-by: timyhac <[email protected]>
timyhac added 8 commits July 16, 2024 18:55
The nuget.config file is only used for testing the
libplctag.NativeImport package, and the rest of the logic for those
tests is in the Build.cs file, so I decided to just combine the two.
So that we can run this workflow which is only defined on the branch, without merging to master

Signed-off-by: timyhac <[email protected]>
@timyhac timyhac merged commit d52cd94 into master Jul 16, 2024
2 checks passed
@timyhac timyhac deleted the #137 branch July 16, 2024 21:15
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