steve/databricks-cli
steve
/
databricks-cli
mirror of https://github.com/databricks/cli.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

add some more todos:

This commit is contained in:
Shreyas Goenka 2024-08-20 15:39:31 +02:00
parent 75f252e51c
commit 43325fdd0a
No known key found for this signature in database
GPG Key ID: 92A07DF49CCB0622
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
bundle/schema/schema.go
View File

@ -50,7 +50,8 @@ func New(golangType reflect.Type, docs *Docs) (*jsonschema.Schema, error) {
s,
{
Type: jsonschema.StringType,
// TODO:
// TODO: Narrow down the scope of the regex match.
// Also likely need to rename this variable.
Pattern: dynvar.VariableRegex,
},
},

11
libs/jsonschema/from_type.go
View File

@ -62,11 +62,10 @@ func FromType(typ reflect.Type, opts FromTypeOptions) (Schema, error) {
res = Schema{Type: StringType}
case reflect.Bool:
res = Schema{Type: BooleanType}
// case reflect.Int, reflect.Int32, reflect.Int64:
// res = Schema{Type: IntegerType}
case reflect.Int, reflect.Int8, reflect.Int16, reflect.Int32, reflect.Int64,
reflect.Uint, reflect.Uint8, reflect.Uint16, reflect.Uint32, reflect.Uint64,
reflect.Float32, reflect.Float64:
// TODO: Add comment about reduced coverage of primitive Go types in the code paths here.
case reflect.Int:
res = Schema{Type: IntegerType}
case reflect.Float32, reflect.Float64:
res = Schema{Type: NumberType}
default:
return InvalidSchema, fmt.Errorf("unsupported type: %s", typ.Kind())
@ -164,6 +163,8 @@ func fromTypeStruct(typ reflect.Type, opts FromTypeOptions) (Schema, error) {
return res, nil
}
// TODO: Add comments explaining the translation between struct, map, slice and
// the JSON schema representation.
func fromTypeSlice(typ reflect.Type, opts FromTypeOptions) (Schema, error) {
if typ.Kind() != reflect.Slice {
return InvalidSchema, fmt.Errorf("expected slice, got %s", typ.Kind())