Skip to content

Commit

Permalink
Remove os.lookup_env which is mostly redundant and a bit deficient
Browse files Browse the repository at this point in the history
  • Loading branch information
myzie committed Aug 26, 2023
1 parent 41c8aca commit 75a30ff
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions modules/os/os.go
Original file line number Diff line number Diff line change
Expand Up @@ -221,21 +221,6 @@ func Unsetenv(ctx context.Context, args ...object.Object) object.Object {
return object.Nil
}

func LookupEnv(ctx context.Context, args ...object.Object) object.Object {
if err := arg.Require("os.lookup_env", 1, args); err != nil {
return err
}
key, err := object.AsString(args[0])
if err != nil {
return err
}
value, ok := GetOS(ctx).LookupEnv(key)
return object.NewMap(map[string]object.Object{
"value": object.NewString(value),
"exists": object.NewBool(ok),
})
}

func ReadFile(ctx context.Context, args ...object.Object) object.Object {
if err := arg.Require("os.read_file", 1, args); err != nil {
return err
Expand Down Expand Up @@ -500,7 +485,6 @@ func Module() *object.Module {
"getuid": object.NewBuiltin("getuid", Getuid),
"getwd": object.NewBuiltin("getwd", Getwd),
"hostname": object.NewBuiltin("hostname", Hostname),
"lookup_env": object.NewBuiltin("lookup_env", LookupEnv),
"mkdir_all": object.NewBuiltin("mkdir_all", MkdirAll),
"mkdir_temp": object.NewBuiltin("mkdir_temp", MkdirTemp),
"mkdir": object.NewBuiltin("mkdir", Mkdir),
Expand Down

0 comments on commit 75a30ff

Please sign in to comment.