Skip to content

Commit

Permalink
cmd/cue: remove mention of CUE_EXPERIMENT=modules from help
Browse files Browse the repository at this point in the history
It doesn't make sense to mention the experiment when it cannot
be disabled.

Signed-off-by: Roger Peppe <[email protected]>
Change-Id: I1ad2a9eaa2d9b99a4e265734ce1286e442aeea53
Dispatch-Trailer: {"type":"trybot","CL":1200582,"patchset":1,"ref":"refs/changes/82/1200582/1","targetBranch":"master"}
  • Loading branch information
rogpeppe authored and cueckoo committed Sep 3, 2024
1 parent 0ab4ab9 commit 470fad3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
3 changes: 0 additions & 3 deletions cmd/cue/cmd/help.go
Original file line number Diff line number Diff line change
Expand Up @@ -312,9 +312,6 @@ If an environment variable is unset or empty, sensible default setting is used.
CUE_EXPERIMENT
Comma-separated list of experiment flags to enable or disable:
modules (default true)
Enable support for the Modules and package management proposal
as described in https://cuelang.org/discussion/2939.
evalv3
Enable the new CUE evaluator, addressing performance issues
and bringing a better disjunction algorithm.
Expand Down
8 changes: 3 additions & 5 deletions cue/load/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -288,11 +288,9 @@ type Config struct {

// Registry is used to fetch CUE module dependencies.
//
// When nil, if the modules experiment is enabled
// (CUE_EXPERIMENT=modules), [modconfig.NewRegistry]
// will be used to create a registry instance using the
// usual cmd/cue conventions for environment variables
// (but see the Env field below).
// When nil, [modconfig.NewRegistry] will be used to create a
// registry instance using the usual cmd/cue conventions for
// environment variables (but see the Env field below).
//
// THIS IS EXPERIMENTAL. API MIGHT CHANGE.
Registry modconfig.Registry
Expand Down

0 comments on commit 470fad3

Please sign in to comment.