Move package celext out of internal (#56)

Move `celext` out of internal so that `buf lint` can import this package and load `DefaultEnv` before checking if a CEL expression compiles.
bufbuild · Oct 2, 2023 · 9023bd7 · 9023bd7
1 parent e5ffd4a
commit 9023bd7
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 7 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -67,7 +67,7 @@ issues:
         - varnamelen
         - unparam
         - gosec
-    - path: internal/celext/lib.go
+    - path: celext/lib.go
       linters:
         # setting up custom functions/overloads appears duplicative (false positive)
         - dupl

diff --git a/internal/celext/lib.go → celext/lib.go b/internal/celext/lib.go → celext/lib.go
@@ -33,8 +33,7 @@ import (
 // DefaultEnv produces a cel.Env with the necessary cel.EnvOption and
 // cel.ProgramOption values preconfigured for usage throughout the
 // module. If useUTC is true, timestamp operations use the UTC timezone instead
-// of the local timezone. If locale is non-empty, the provided locale string is
-// used for string formatting, defaulting to 'en_US' if unset.
+// of the local timezone.
 func DefaultEnv(useUTC bool) (*cel.Env, error) {
 	return cel.NewEnv(
 		cel.Lib(lib{

diff --git a/internal/celext/lib_test.go → celext/lib_test.go b/internal/celext/lib_test.go → celext/lib_test.go
diff --git a/internal/constraints/cache_test.go b/internal/constraints/cache_test.go
@@ -18,7 +18,7 @@ import (
 	"testing"
 
 	"buf.build/gen/go/bufbuild/protovalidate/protocolbuffers/go/buf/validate"
-	"github.com/bufbuild/protovalidate-go/internal/celext"
+	"github.com/bufbuild/protovalidate-go/celext"
 	"github.com/bufbuild/protovalidate-go/internal/gen/buf/validate/conformance/cases"
 	"github.com/google/cel-go/cel"
 	"github.com/stretchr/testify/assert"

diff --git a/internal/evaluator/builder_test.go b/internal/evaluator/builder_test.go
@@ -18,7 +18,7 @@ import (
 	"sync"
 	"testing"
 
-	"github.com/bufbuild/protovalidate-go/internal/celext"
+	"github.com/bufbuild/protovalidate-go/celext"
 	pb "github.com/bufbuild/protovalidate-go/internal/gen/tests/example/v1"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"

diff --git a/internal/expression/ast_test.go b/internal/expression/ast_test.go
@@ -18,7 +18,7 @@ import (
 	"testing"
 
 	"buf.build/gen/go/bufbuild/protovalidate/protocolbuffers/go/buf/validate"
-	"github.com/bufbuild/protovalidate-go/internal/celext"
+	"github.com/bufbuild/protovalidate-go/celext"
 	"github.com/google/cel-go/cel"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"

diff --git a/validator.go b/validator.go
@@ -18,7 +18,7 @@ import (
 	"fmt"
 
 	"buf.build/gen/go/bufbuild/protovalidate/protocolbuffers/go/buf/validate"
-	"github.com/bufbuild/protovalidate-go/internal/celext"
+	"github.com/bufbuild/protovalidate-go/celext"
 	"github.com/bufbuild/protovalidate-go/internal/errors"
 	"github.com/bufbuild/protovalidate-go/internal/evaluator"
 	"google.golang.org/protobuf/proto"