Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apply go vet and golint #37

Merged
merged 12 commits into from
Oct 12, 2017
Merged

Apply go vet and golint #37

merged 12 commits into from
Oct 12, 2017

Conversation

sue445
Copy link
Owner

@sue445 sue445 commented Oct 12, 2017

No description provided.

@sue445 sue445 changed the title Apply go vet and golint [WIP] Apply go vet and golint Oct 12, 2017
```
chatwork_notifier.go:10:6: exported type ChatworkNotifier should have comment or be unexported
chatwork_notifier.go:15:1: exported function NewChatworkNotifier should have comment or be unexported
chatwork_notifier.go:22:1: exported method ChatworkNotifier.ExpectedKeys should have comment or be unexported
chatwork_notifier.go:26:1: exported method ChatworkNotifier.PostStatus should have comment or be unexported
config.go:8:6: exported type Config should have comment or be unexported
config.go:10:1: exported function LoadConfigFromData should have comment or be unexported
config.go:21:1: exported function LoadConfigFromFile should have comment or be unexported
main.go:10:2: exported var Version should have comment or be unexported
notifier.go:3:6: exported type PostStatusParam should have comment or be unexported
notifier.go:10:6: exported type Notifier should have comment or be unexported
slack_notifier.go:10:6: exported type SlackNotifier should have comment or be unexported
slack_notifier.go:16:1: exported function NewSlackNotifier should have comment or be unexported
slack_notifier.go:30:1: exported method SlackNotifier.ExpectedKeys should have comment or be unexported
slack_notifier.go:34:1: exported method SlackNotifier.PostStatus should have comment or be unexported
status_store.go:9:6: exported type StatusStore should have comment or be unexported
status_store.go:14:2: exported const NOT_FOUND_KEY should have comment (or a comment on this block) or be unexported
status_store.go:17:1: exported function NewStatusStore should have comment or be unexported
status_store.go:23:1: exported method StatusStore.GetDbStatus should have comment or be unexported
status_store.go:42:1: exported method StatusStore.SaveDbStatus should have comment or be unexported
util.go:5:1: exported function GetStatusCode should have comment or be unexported
util.go:16:1: exported function IsSuccessfulStatus should have comment or be unexported
```
@sue445 sue445 changed the title [WIP] Apply go vet and golint Apply go vet and golint Oct 12, 2017
@sue445 sue445 merged commit 3f79899 into master Oct 12, 2017
@sue445 sue445 deleted the feature/golint branch October 12, 2017 15:10
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