Conversation
|
✅ No conflicts with other open PRs targeting |
|
There was a problem hiding this comment.
Pull request overview
Risk Rating: MEDIUM (core dependency bump; behavioral changes are primarily in the upstream libocr PR)
This PR updates the repository’s Go modules to use a newer github.com/smartcontractkit/libocr pseudo-version (per OEV-988 / libocr#133), and records the change via a Changeset.
Changes:
- Bump
github.com/smartcontractkit/libocrfromv0.0.0-20260130195252-6e18e2a30acctov0.0.0-20260304194147-a03701e2c02eacross multiple Go modules. - Update corresponding
go.sumentries to match the new module version/hashes. - Add a Changeset entry for the bump.
Areas needing scrupulous human review:
- The upstream libocr change set (libocr PR #133 / diff link in PR description), since this repo change is a version pointer update and all functional risk is in the dependency delta.
- Smoke/integration/system test execution for OCR/OCR2-related flows impacted by libocr.
Suggested reviewers (per CODEOWNERS):
go.mod,go.sum:@smartcontractkit/core@smartcontractkit/foundationsintegration-tests/*:@smartcontractkit/core@smartcontractkit/devex-tooling@smartcontractkit/foundations.changeset/*:@smartcontractkit/foundations@smartcontractkit/core/deployment/*:@smartcontractkit/core(plus the listed deployment owners, depending on release impact)
Reviewed changes
Copilot reviewed 8 out of 15 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| system-tests/tests/go.mod | Bumps libocr version for system test module. |
| system-tests/tests/go.sum | Updates libocr checksums for system test module. |
| system-tests/lib/go.mod | Bumps libocr version for system test shared lib. |
| system-tests/lib/go.sum | Updates libocr checksums for system test shared lib. |
| integration-tests/load/go.mod | Updates indirect libocr version for load tests module. |
| integration-tests/load/go.sum | Updates libocr checksums for load tests module. |
| integration-tests/go.mod | Bumps libocr version for integration tests module. |
| integration-tests/go.sum | Updates libocr checksums for integration tests module. |
| go.mod | Bumps root module libocr dependency. |
| go.sum | Updates root module libocr checksums. |
| deployment/go.mod | Bumps libocr version for deployment tooling module. |
| deployment/go.sum | Updates libocr checksums for deployment tooling module. |
| core/scripts/go.mod | Bumps libocr version for core scripts module. |
| core/scripts/go.sum | Updates libocr checksums for core scripts module. |
| .changeset/late-wolves-feel.md | Adds Changeset describing the libocr bump. |
| "chainlink": minor | ||
| --- | ||
|
|
||
| #updated Bump libocr |
There was a problem hiding this comment.
The changeset summary line uses the #updated tag, but this repo’s changesets appear to use the first non-frontmatter line as a human-readable summary (and when tags are used, they’re separate lines like #internal). As written, #updated Bump libocr will be emitted verbatim into the changelog; consider changing this to a normal summary (e.g., Bump libocr) and, if needed, add a separate standardized tag line (such as #internal).
| #updated Bump libocr | |
| Bump libocr |
OEV-988 Bump libocr (#21411)
OEV-988
Requires
smartcontractkit/libocr#133
Libocr diff
smartcontractkit/libocr@6e18e2a...a03701e