Skip to content

HYPERFLEET-1578 - fix: migrate cel-go to renamed cel.dev/cel-go path - #255

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-hyperfleet:mainfrom
rafabene:HYPERFLEET-1578-migrate-cel-go-module-path
Aug 25, 2026
Merged

HYPERFLEET-1578 - fix: migrate cel-go to renamed cel.dev/cel-go path#255
openshift-merge-bot[bot] merged 1 commit into
openshift-hyperfleet:mainfrom
rafabene:HYPERFLEET-1578-migrate-cel-go-module-path

Conversation

@rafabene

Copy link
Copy Markdown
Member

Summary

  • Migrates the CEL dependency from github.com/google/cel-go to its renamed module path cel.dev/cel-go (v0.32.0)
  • Rewrites imports in internal/payload/builder.go and internal/engine/decision.go
  • No behavior change — same package API, only the import path changed

Why

Starting at v0.32.0, google/cel-go renamed its Go module path to cel.dev/cel-go, which Renovate cannot rewrite automatically. This was blocking dependency-update PRs (#253) on the renovate/artifacts check with:

```text
go: github.com/google/cel-go@v0.32.0: parsing go.mod:
module declares its path as: cel.dev/cel-go
but was required as: github.com/google/cel-go
```

Jira: https://redhat.atlassian.net/browse/HYPERFLEET-1578

Test plan

  • make test — unit tests pass
  • make test-integration — integration tests pass
  • make lint — no issues
  • Verified no remaining references to github.com/google/cel-go in code or docs

Starting at v0.32.0, google/cel-go renamed its Go module path to
cel.dev/cel-go, which Renovate cannot rewrite automatically and was
blocking dependency-update PRs (renovate/artifacts check failing on
go get -t ./...). Updates go.mod and rewrites all imports in
internal/payload/builder.go and internal/engine/decision.go.
@hyperfleet-ci-bot

Copy link
Copy Markdown

Risk Score: 2 — risk/medium

Signal Detail Points
PR size 17 lines +0
Sensitive paths none +0
Test coverage No _test.go files in diff +2

Computed by hyperfleet-risk-scorer

@pnguyen44 pnguyen44 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci

openshift-ci Bot commented Aug 25, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pnguyen44

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit eaf1686 into openshift-hyperfleet:main Aug 25, 2026
9 checks passed
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