Skip to content

[jsonschema/schema.go] Improve error handling#64

Open
SamuelMarks wants to merge 1 commit intogoogle:mainfrom
fork-google:improve-error-handling
Open

[jsonschema/schema.go] Improve error handling#64
SamuelMarks wants to merge 1 commit intogoogle:mainfrom
fork-google:improve-error-handling

Conversation

@SamuelMarks
Copy link

I used the popular errcheck tool to find unhandled errors:

$ errcheck ./...                                                          
jsonschema/resolve.go:448:15:  setAnchor(s, baseInfo, anchorName, false)        
jsonschema/resolve.go:463:13:  setAnchor(s, baseInfo, s.Anchor, false)        
jsonschema/resolve.go:464:13:  setAnchor(s, baseInfo, s.DynamicAnchor, true)

Then I prepared this open-source [Apache-2.0] offline developer tool go-auto-err-handling to automatically add the if err := …; err != nil { return err } lines.

Thanks for reviewing and 🤞 merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant