Skip to content

Conversation

@amotl
Copy link
Member

@amotl amotl commented Nov 15, 2025

About

Just maintenance.

Details

Currently we need to do the version bumping here manually, so relevant changes need to accompany corresponding signals by Dependabot, because the current architecture how Npgsql versions are populated from the test matrix definition requires it.

npgsql-version: [ '8.0.7', '9.0.3' ]

- name: Validate by-language/csharp-npgsql, Npgsql ${{ matrix.npgsql-version }}
run: |
uv run --with=pueblo ngr test by-language/csharp-npgsql --dotnet-version=${{ matrix.dotnet-version }} --npgsql-version=${{ matrix.npgsql-version }}

@coderabbitai
Copy link

coderabbitai bot commented Nov 15, 2025

Walkthrough

Updated the CI workflow's Npgsql test matrix from versions 8.0.7 and 9.0.3 to 8.0.8, 9.0.4, and 10.0.0-rc.1, with a documentation comment added. This expands test coverage to include a pre-release and additional version.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/lang-csharp-npgsql.yml
Updated test matrix Npgsql versions and added documentation comment

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Versions dance and march on through,
Eight-point-oh and ten so new,
Pre-release arrives to test the way,
CI pipelines work and play! 🚀

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating CI validation to test against specific Npgsql versions (8.0.8, 9.0.4, and 10.0.0-rc.1).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description clearly explains that this is maintenance work updating Npgsql versions in the CI matrix, directly related to the changeset.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch npgsql-202511-updates

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@amotl amotl marked this pull request as ready for review November 15, 2025 10:11
@amotl amotl requested review from seut and surister November 15, 2025 10:12
@amotl
Copy link
Member Author

amotl commented Nov 15, 2025

Currently we need to do the version bumping here manually [...]

Discarding this in favor of a recipe that needs less maintenance.

@amotl amotl closed this Nov 15, 2025
@amotl amotl deleted the npgsql-202511-updates branch November 15, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants