NuGet package dependency with version defined as msbuild property not getting updated #33318
Unanswered
AArnott
asked this question in
Request Help
Replies: 3 comments 5 replies
-
Please edit and remove the mentions of dependabot, it's unnecessary to do that in order to request a feature |
Beta Was this translation helpful? Give feedback.
3 replies
-
#29420 |
Beta Was this translation helpful? Give feedback.
1 reply
-
If the variable is in the same file then we should be able to support this easily. Would leverage the "sharedVariableName" refactor I had in another PR, to resume in the new year |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
How are you running Renovate?
A Mend.io-hosted app
If you're self-hosting Renovate, tell us which platform (GitHub, GitLab, etc) and which version of Renovate.
No response
Please tell us more about your question or problem
The NuGet package dependency expressed here (and whose referenced msbuild property is defined earlier in the file) is not getting an update PR created by Renovate.
The renovate logs mention most of the other package names that I spot checked, but "PolyType" does not appear at all, leading me to believe that somehow the scanner is missing PolyType as a dependency.
I then checked for "Microsoft.CodeAnalysis.CSharp.Workspaces" in the log and couldn't find that either. This dependency is also based on an msbuild property.
This leads me to believe that Renovate fails whenever the package version isn't a literal number but rather an msbuild property that is set to a version number.
If so, this is a loss for me in functionality since switching to Renovate from (redacted) earlier this week, which notices when groups of package versions are defined by the same property and will issue an update to the property, and then in the PR title it references an update of the property.
Now, if/when Renovate supports this, it could certainly do better than the competition by:
Logs (if relevant)
Logs
Beta Was this translation helpful? Give feedback.
All reactions