Skip to content

Commit

Permalink
Bypass deprecation lint warning (#45)
Browse files Browse the repository at this point in the history
CI is failing because of a deprecation notice on
`cel.OptCheckStringFormat`, but the recommended alternative doesn't
appear to exist. I've filed google/cel-go#837
upstream to clarify, but for now tests are passing so I think we can
safely bypass this lint error.

I also filed #44 so
that we don't forget to migrate to whichever API is recommended and
remove this bypass.
  • Loading branch information
akshayjshah committed Sep 12, 2023
1 parent c50cc86 commit 33692a1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/celext/lib.go
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,7 @@ func (l lib) ProgramOptions() []cel.ProgramOption {
return []cel.ProgramOption{
cel.EvalOptions(
cel.OptOptimize,
cel.OptCheckStringFormat,
cel.OptCheckStringFormat, //nolint:staticcheck
),
}
}
Expand Down

0 comments on commit 33692a1

Please sign in to comment.