-
-
Notifications
You must be signed in to change notification settings - Fork 2
Add platform-agnostic portable deployment as ZIP attachment to releases #125
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
Conversation
Co-authored-by: dennisdoomen <[email protected]>
Co-authored-by: dennisdoomen <[email protected]>
Co-authored-by: dennisdoomen <[email protected]>
Co-authored-by: dennisdoomen <[email protected]>
Co-authored-by: dennisdoomen <[email protected]>
Co-authored-by: dennisdoomen <[email protected]>
Build/Build.cs
Outdated
| DotNetPublish(s => s | ||
| .SetProject(Solution.PackageGuard) | ||
| .SetConfiguration(Configuration) | ||
| .SetRuntime("win-x64") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep the assemblies platform-agnostic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot please address this comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to platform-agnostic framework-dependent deployment. Removed runtime-specific settings (win-x64, self-contained, single-file) so the assemblies now work on Windows, Linux, and macOS with .NET 9.0+. See commit 88d5338.
Co-authored-by: dennisdoomen <[email protected]>
|
@copilot please rebase this PR and resolve the merge conflicts |
I've successfully rebased the PR onto the latest main branch (commit 0712715) and resolved all merge conflicts in README.md. The platform-agnostic portable deployment changes are preserved while incorporating the new NPM/Yarn/PNPM support from main. However, I'm unable to force-push the rebased commits due to authentication limitations. The rebased branch is ready locally with all 8 commits cleanly applied on top of main. Could you please verify if the remote branch was updated, or let me know if you need me to take a different approach to complete the rebase? Local branch status:
|
Pull Request Test Coverage Report for Build 19136752438Details
💛 - Coveralls |
Enables users to run PackageGuard on any platform (Windows, Linux, macOS) by providing a portable deployment in release assets.
Build System
PublishBinarytarget that publishes platform-agnostic framework-dependent deploymentPackageGuard-{version}.zipcontaining DLL files and all dependencies.OnlyWhenDynamic(() => IsTag)CI/CD
softprops/action-gh-release@v2Documentation
packageguardanddotnet PackageGuard.dllinvocationsExample usage after downloading release ZIP:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.