deps: Update ancplua-sdk#199
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7b481bdcb2
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "ANcpLua.Roslyn.Utilities.Testing": "2.2.41", | ||
| "ANcpLua.Roslyn.Utilities": "2.2.41" |
There was a problem hiding this comment.
With these MSBuild SDK pins at 2.2.41, Directory.Packages.props still resolves the actual PackageReferences through Version.props, where the Roslyn.Utilities and Testing variables remain 2.2.39. The repo contract says local overrides win and must be deleted once they are equal to or below SDK truth, so restores/packing will mix SDK/build assets from 2.2.41 with runtime/testing packages from 2.2.39 instead of consuming this dependency update; please bump or remove those overrides in the same change.
Useful? React with 👍 / 👎.
7b481bd to
c158e30
Compare
renovate
Bot
changed the title
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c158e30ae4
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "ANcpLua.Roslyn.Utilities.Testing": "2.2.42", | ||
| "ANcpLua.Roslyn.Utilities": "2.2.42" |
There was a problem hiding this comment.
Fresh evidence in this revision is that the MSBuild SDK pins now move both Roslyn.Utilities SDKs to 2.2.42, but Directory.Packages.props still imports the local Version.props variables where the actual ANcpLua.Roslyn.Utilities and ANcpLua.Roslyn.Utilities.Testing PackageVersions remain 2.2.39. Restores and packs will keep using the stale runtime/test packages instead of consuming this dependency bump, so please remove or update those local overrides with the SDK bump.
Useful? React with 👍 / 👎.
c158e30 to
39727c8
Compare
renovate
Bot
changed the title
This PR contains the following updates:
3.4.44→3.4.453.4.44→3.4.453.4.44→3.4.452.2.40→2.2.422.2.40→2.2.42Release Notes
ANcpLua/ANcpLua.NET.Sdk (ANcpLua.NET.Sdk)
v3.4.45What's Changed
Full Changelog: ANcpLua/ANcpLua.NET.Sdk@v3.4.44...v3.4.45
ANcpLua/ANcpLua.Roslyn.Utilities (ANcpLua.Roslyn.Utilities)
v2.2.42Full Changelog: ANcpLua/ANcpLua.Roslyn.Utilities@v2.2.41...v2.2.42
v2.2.41What's Changed
Full Changelog: ANcpLua/ANcpLua.Roslyn.Utilities@v2.2.40...v2.2.41
Configuration
📅 Schedule: (in timezone UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.