Skip to content

Commit

Permalink
Change default scope to DeclScope
Browse files Browse the repository at this point in the history
The `README` says the default scope is `declarations` but the code currently sets it to `toplevel`.

So one or the other needs to be adjusted so they are in-sync.

We have hit a bunch of false positives with `toplevel` and so would prefer to keep the default at `declarations`.

Especially since it's painful to change just that setting as it requires a config file since it's not possible to pass in a dict/map of custom settings as a CLI arg:
#17
  • Loading branch information
jeffwidman committed Apr 8, 2021
1 parent 5996335 commit c42c547
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/godot/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ var version = "master"
const defaultConfigFile = "config.yaml"

var defaultSettings = godot.Settings{
Scope: godot.TopLevelScope,
Scope: godot.DeclScope,
Period: true,
Capital: false,
}
Expand Down

0 comments on commit c42c547

Please sign in to comment.