lourenco/mgmt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

lang: Rephrase the scope building log messages

Browse Source
This commit is contained in:
James Shubin
2024-03-13 17:00:02 -04:00
parent e0ace35525
commit f973009b83
3 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lang/ast/structs.go
View File

@@ -3480,7 +3480,7 @@ func (obj *StmtProg) importSystemScope(name string) (*interfaces.Scope, error) {
return nil, errwrap.Wrapf(err, "could not interpolate AST from import `%s`", name) return nil, errwrap.Wrapf(err, "could not interpolate AST from import `%s`", name)
} }
obj.data.Logf("building scope...") obj.data.Logf("scope building...")
// propagate the scope down through the AST... // propagate the scope down through the AST...
// most importantly, we ensure that the child imports will run! // most importantly, we ensure that the child imports will run!
// we pass in *our* parent scope, which will include the globals // we pass in *our* parent scope, which will include the globals
@@ -3619,7 +3619,7 @@ func (obj *StmtProg) importScopeWithParsedInputs(input *inputs.ParsedInput, scop
return nil, errwrap.Wrapf(err, "could not interpolate AST from import") return nil, errwrap.Wrapf(err, "could not interpolate AST from import")
} }
logf("building scope...") logf("scope building...")
// propagate the scope down through the AST... // propagate the scope down through the AST...
// most importantly, we ensure that the child imports will run! // most importantly, we ensure that the child imports will run!
// we pass in *our* parent scope, which will include the globals // we pass in *our* parent scope, which will include the globals

2
lang/gapi/gapi.go
View File

@@ -245,7 +245,7 @@ func (obj *GAPI) Cli(info *gapi.Info) (*gapi.Deploy, error) {
Functions: ast.FuncPrefixToFunctionsScope(""), // runs funcs.LookupPrefix Functions: ast.FuncPrefixToFunctionsScope(""), // runs funcs.LookupPrefix
} }
logf("building scope...") logf("scope building...")
// propagate the scope down through the AST... // propagate the scope down through the AST...
// We use SetScope because it follows all of the imports through. I did // We use SetScope because it follows all of the imports through. I did
// not think we needed to pass in an initial scope because the download // not think we needed to pass in an initial scope because the download

2
lang/lang.go
View File

@@ -205,7 +205,7 @@ func (obj *Lang) Init() error {
Functions: ast.FuncPrefixToFunctionsScope(""), // runs funcs.LookupPrefix Functions: ast.FuncPrefixToFunctionsScope(""), // runs funcs.LookupPrefix
} }
obj.Logf("building scope...") obj.Logf("scope building...")
// propagate the scope down through the AST... // propagate the scope down through the AST...
if err := obj.ast.SetScope(scope); err != nil { if err := obj.ast.SetScope(scope); err != nil {
return errwrap.Wrapf(err, "could not set scope") return errwrap.Wrapf(err, "could not set scope")