-
-
Notifications
You must be signed in to change notification settings - Fork 731
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
Add PublishReadyToRun to DotNetRestoreSettings #4109
Closed
Labels
Milestone
Comments
Is it up for contribution? |
Hey @SARAVANA1501, no this one already has a pull-request that is waiting to be merged - #4110 |
Assigned to @Marusyk |
augustoproiete
pushed a commit
to Marusyk/cake
that referenced
this issue
Apr 4, 2023
devlead
pushed a commit
to Marusyk/cake
that referenced
this issue
Jun 29, 2023
devlead
added a commit
that referenced
this issue
Jun 29, 2023
GH-4109: Add PublishReadyToRun to DotNetRestoreSettings
🎉 This issue has been resolved in version v3.1.0 🎉 The release is available on: Your GitReleaseManager bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Discussed in https://github.com/orgs/cake-build/discussions/4108
Originally posted by Trisped January 19, 2023
According to https://learn.microsoft.com/en-us/dotnet/core/compatibility/sdk/6.0/publish-readytorun-requires-restore-change#recommended-action :
According to https://cakebuild.net/api/Cake.Common.Tools.DotNet.Restore/DotNetRestoreSettings/ , there is not a PublishReadyToRun property like there is in DotNetPublishSettings.
There is a work around (set NoRestore to false), but it requires restoring twice, if you have already restored (for example, so you could test before publishing).
The text was updated successfully, but these errors were encountered: