Skip to content

build(deps): bump ANcpLua.Analyzers 1.29.0 → 1.29.1#92

Merged
ANcpLua merged 1 commit intomainfrom
bump-analyzers-1.29.1
May 4, 2026
Merged

build(deps): bump ANcpLua.Analyzers 1.29.0 → 1.29.1#92
ANcpLua merged 1 commit intomainfrom
bump-analyzers-1.29.1

Conversation

@ANcpLua
Copy link
Copy Markdown
Owner

@ANcpLua ANcpLua commented May 4, 2026

Summary

  • Bump ANcpLuaAnalyzersVersion in Truth (Version.props) from 1.29.0 to 1.29.1 to match latest published NuGet version
  • Verified: https://api.nuget.org/v3-flatcontainer/ancplua.analyzers/index.json lists 1.29.1

Test plan

  • CI green (ubuntu/windows/macos)
  • Verify NuGet has 1.29.1 before bump (no NU1102 risk)

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 4, 2026 09:57
@ANcpLua ANcpLua enabled auto-merge (squash) May 4, 2026 09:57
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: f4bff418-7752-478d-b173-f3215b00edc9

📥 Commits

Reviewing files that changed from the base of the PR and between 4a29ee8 and e867e2c.

📒 Files selected for processing (1)
  • src/Build/Common/Version.props
📜 Recent review details
⏰ Context from checks skipped due to timeout of 120000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: create_nuget
  • GitHub Check: Agent
  • GitHub Check: lint_config
🧰 Additional context used
📓 Path-based instructions (1)
**/*.props

⚙️ CodeRabbit configuration file

MSBuild property files (Directory.Build.props, Directory.Packages.props, Version.props). Review for: Central Package Management correctness, version consistency, and that new packages are added with explicit version pins. Flag transitive dependency promotions that aren't justified. Verify TFM targeting is correct (.NET 10).

Files:

  • src/Build/Common/Version.props
🔇 Additional comments (1)
src/Build/Common/Version.props (1)

56-56: ANcpLua.Analyzers version 1.29.1 is published on NuGet with no security advisories. Change is valid.


Summary by CodeRabbit

  • Chores
    • Updated version to 1.29.1

Walkthrough

The ANcpLuaAnalyzersVersion property in the build configuration is incremented from 1.29.0 to 1.29.1, updating the NCP Lua Analyzers dependency version.

Changes

Version Update

Layer / File(s) Summary
Analyzer Version Bump
src/Build/Common/Version.props
ANcpLuaAnalyzersVersion incremented to 1.29.1.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Suggested labels

area:infra

🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Title check ✅ Passed Title follows conventional commits format with 'build' prefix, includes scope 'deps', and is well under 72 characters.
Description check ✅ Passed Description clearly relates to the changeset—it documents the version bump with NuGet verification and test coverage.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Otel Instrumentation Required ✅ Passed PR contains only a NuGet package version bump in a props file with no new injectable service classes or DI registrations.
No Unbounded Mcp Responses ✅ Passed PR contains only a version property bump in Version.props with no MCP tool definitions or modifications.
Duckdb Backpressure On Write Paths ✅ Passed PR contains only a version property update in build configuration with no functional changes, code additions, or database interactions.
Cancellationtoken Threading ✅ Passed PR modifies only build configuration file; no new public async methods in C# source code.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Review rate limit: 4/5 reviews remaining, refill in 12 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the ANcpLuaAnalyzersVersion from 1.29.0 to 1.29.1 in the Version.props file. I have no feedback to provide as there are no review comments and the change is a straightforward version bump.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s centrally managed build versions to reference the latest published ANcpLua.Analyzers package, ensuring the SDK’s analyzer dependencies stay in sync with the NuGet feed.

Changes:

  • Bump ANcpLuaAnalyzersVersion from 1.29.0 to 1.29.1 in Version.props.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ANcpLua ANcpLua merged commit b9dc17e into main May 4, 2026
21 checks passed
@ANcpLua ANcpLua deleted the bump-analyzers-1.29.1 branch May 4, 2026 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants