Skip to content

#141 bump senzing to 3.13.2 - #142

Merged
kernelsam merged 1 commit into
mainfrom
141-skern
Aug 5, 2026
Merged

#141 bump senzing to 3.13.2#142
kernelsam merged 1 commit into
mainfrom
141-skern

Conversation

@kernelsam

@kernelsam kernelsam commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Pull request questions

Which issue does this address

Issue number: #141


Resolves #141

@kernelsam
kernelsam requested review from a team as code owners August 5, 2026 16:24
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

🤖 Claude Code Review

Senzing Code Review

Scope: Version-bump PR updating the base image (debian:13.3-slim13.5-slim), the Senzing runtime package (3.13.1-253233.13.2-26210), plus a Renovate config migration and CHANGELOG entry. This is a routine dependency-bump PR — no application code changed.

Code Quality

  • Style conventionsDockerfile and .github/renovate.json changes are consistent with existing formatting/idioms in the repo.
  • No commented-out code
  • Meaningful variable names — N/A, only ARG/ENV value bumps.
  • DRY — N/A for this diff.
  • ⚠️ Defects.github/renovate.json:13 migrates matchPackagePrefixes: ["!senzing/"] to matchPackageNames: ["*", "!senzing/**"]. This is the correct modern Renovate syntax (the old matchPackagePrefixes option is deprecated in favor of glob-capable matchPackageNames), and the intent (apply the 21-day minimumReleaseAge delay to everything except senzing/* packages) is preserved. One subtle difference worth double-checking manually (no network access to verify against live Renovate docs): senzing/** matches nested paths (e.g. senzing/foo/bar) whereas the old prefix-match senzing/ matched on string prefix regardless of path depth — functionally near-identical for this repo's actual package set, so low risk.
  • ✅ No .claude/CLAUDE.md exists in this repo (only in the untracked build-resources/ checkout, which is not part of this PR/diff and appears to be a local scratch checkout, not a repo file being added).

Testing

  • N/A — this PR only bumps version pins/labels in Dockerfile and updates config/changelog; no new functions or endpoints. Nothing to unit/integration test beyond CI's existing image-build pipeline.

Documentation

  • CHANGELOG.md updatedCHANGELOG.md:10-15 adds a [3.13.2] - 2026-08-05 entry following the same structure/style as prior entries (### Changed in 3.13.2, matches Keep a Changelog format used throughout the file).
  • CommonMark/formatting — no stray whitespace, headings and list formatting match the rest of the file; consistent with prettier-style output.
  • REFRESHED_AT label bumped (Dockerfile:11, 2026-06-172026-08-05), matching the CI check referenced in build-resources/.github/workflows/verify-dockerfile-refreshed-at-updated.yaml.
  • N/A README/API docs — no user-facing behavior changed.

Security

  • No hardcoded credentials
  • Base image pinned by digestDockerfile:1 updates both the tag (13.3-slim13.5-slim) and the sha256 digest together, avoiding a mismatched/stale digest.
  • No .lic files or AQAAAD-prefixed strings introduced by this diff. I searched the full working tree for .lic files and AQAAAD strings; the only hits are in two untracked local scratch files (prompt-header.md, prompt-temp.md) which merely contain a copy of this checklist's own text (Include files containing strings starting with \AQAAAD``) — not an actual license key, and not part of this PR's diff.
  • ✅ No sensitive data in logs/error handling — N/A, no code paths changed.

Summary

This is a clean, low-risk version-bump PR. The Renovate config change correctly migrates a deprecated matcher to current glob syntax with equivalent intent, the CHANGELOG entry follows house style, and the Dockerfile update pairs the tag bump with its correct digest. No blocking issues found.

Automated code review analyzing defects and coding standards

@kernelsam
kernelsam enabled auto-merge (squash) August 5, 2026 16:34
@kernelsam
kernelsam merged commit 16d8735 into main Aug 5, 2026
13 checks passed
@kernelsam
kernelsam deleted the 141-skern branch August 5, 2026 16:36
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.

update senzing to 3.13.2

2 participants