Addressed style and comment issues

This commit is contained in:
Shreyas Goenka 2023-01-19 15:25:17 +01:00
parent ccefb20e4e
commit 761186680f
No known key found for this signature in database
GPG Key ID: 92A07DF49CCB0622
2 changed files with 20 additions and 26 deletions

View File

@ -1,7 +1,6 @@
package schema package schema
import ( import (
"io/ioutil"
"os" "os"
"gopkg.in/yaml.v2" "gopkg.in/yaml.v2"
@ -13,11 +12,7 @@ type Docs struct {
} }
func LoadDocs(path string) (*Docs, error) { func LoadDocs(path string) (*Docs, error) {
f, err := os.Open(path) bytes, err := os.ReadFile(path)
if err != nil {
return nil, err
}
bytes, err := ioutil.ReadAll(f)
if err != nil { if err != nil {
return nil, err return nil, err
} }

View File

@ -16,23 +16,23 @@ type Schema struct {
// IDE. This is manually injected here using schema.Docs // IDE. This is manually injected here using schema.Docs
Description string `json:"description,omitempty"` Description string `json:"description,omitempty"`
// schemas for the fields of an struct. The keys are the first json tag. // Schemas for the fields of an struct. The keys are the first json tag.
// The values are the schema for the type of the field // The values are the schema for the type of the field
Properties map[string]*Schema `json:"properties,omitempty"` Properties map[string]*Schema `json:"properties,omitempty"`
// the schema for all values of an array // The schema for all values of an array
Items *Schema `json:"items,omitempty"` Items *Schema `json:"items,omitempty"`
// the schema for any properties not mentioned in the Schema.Properties field. // The schema for any properties not mentioned in the Schema.Properties field.
// this validates Maps in bundle configuration // this validates Maps in bundle configuration
// OR // OR
// a boolean type with value false. Setting false here validates that all // A boolean type with value false. Setting false here validates that all
// properties in the yaml file have been defined in the json schema // properties in the yaml file have been defined in the json schema
// //
// Its type during runtime will either be *Schema or bool // Its type during runtime will either be *Schema or bool
AdditionalProperties interface{} `json:"additionalProperties,omitempty"` AdditionalProperties any `json:"additionalProperties,omitempty"`
// required properties for the object. Any fields missing the "omitempty" // Required properties for the object. Any fields missing the "omitempty"
// tag will be included // tag will be included
Required []string `json:"required,omitempty"` Required []string `json:"required,omitempty"`
} }
@ -148,10 +148,9 @@ func safeToSchema(golangType reflect.Type, docs *Docs, debugTraceId string, seen
return props, nil return props, nil
} }
// Adds the member fields of golangType to the passed slice. Needed because // This function returns all member fields of the provided type.
// golangType can contain embedded fields (aka anonymous fields) // If the type has embedded (aka anonymous) fields, this function traverses
// those in a breadth first manner
// The function traverses the embedded fields in a breadth first mannerå
func getStructFields(golangType reflect.Type) []reflect.StructField { func getStructFields(golangType reflect.Type) []reflect.StructField {
fields := []reflect.StructField{} fields := []reflect.StructField{}
bfsQueue := list.New() bfsQueue := list.New()