这是indexloc提供的服务，不要输入任何密码
Skip to content
This repository was archived by the owner on Feb 13, 2025. It is now read-only.

cmd/scollector: error on unused toml keys #1080

Merged
merged 1 commit into from
Jun 18, 2015
Merged

cmd/scollector: error on unused toml keys #1080

merged 1 commit into from
Jun 18, 2015

Conversation

@madelynnblue
Copy link
Contributor

@madelynnblue madelynnblue commented Jun 18, 2015

Review on Reviewable

@gbrayut
Copy link
Contributor

gbrayut commented Jun 18, 2015

Tested with a bogus [[asdfasdf]] and with an extra asdf = "^ChatServer" and it caught both of them. LGTM

2015/06/18 11:15:51 fatal: main.go:146: extra keys in C:\Code\Go\src\bosun.org\cmd\scollector\scollector.toml: [asdfasdf asdfasdf.Name]
...
2015/06/18 11:16:40 fatal: main.go:146: extra keys in C:\Code\Go\src\bosun.org\cmd\scollector\scollector.toml: [ProcessDotNet.asdf]

madelynnblue added a commit that referenced this pull request Jun 18, 2015
@madelynnblue
Merge pull request #1080 from bosun-monitor/toml-strict
deef5c5 
cmd/scollector: error on unused toml keys
@madelynnblue madelynnblue merged commit deef5c5 into master Jun 18, 2015
@madelynnblue madelynnblue deleted the toml-strict branch June 18, 2015 17:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@madelynnblue @gbrayut