Add types and allow empty app ID

[warent]

[mikeflynn]

This change makes sense and is simple enough, but you've got a couple of missing double quotes in your type JSON tags.

[warent]

I'm not sure if I should be more scared that Go never raised an error or that none of my tests picked it up

[harrisonhjones]

@warent Some IDEs (I use GoLand) will call out/highlight malformed tags.

[rking788]

The go vet tool will warn about incorrect struct tags

[mikeflynn]

What's the status on this PR? This conflicts with the current master but if it's still needed, I can merge.

(Also, please don't hesitate to ping me on Twitter [@thatmikeflynn] as my GitHub notifications are impossible to keep up with and for some reason this repo is always at the bottom of the list.)

[harrisonhjones]

@mikeflynn given what @rking788 mentioned how difficult would it be to add a CI check (with go vet) to PRs?

[mikeflynn]

That would be great! Have either of you set that up with GitHub PRs before? Any way you can point me in the right direction?

[rking788]

so I'm not sure if there is a different way that I have never tried before. but the only way I can think of is to setup a travis.ci build for the repo (free for open source projects) and then run go test and go vet in the build script. A sample script can be found in the go-cmp project. Actually now that I am reading the # 86 pull request on that project it sounds like vet is running automatically with go 1.10. so it would just need to be the go test command.

[harrisonhjones]

I can try to setup something on my own fork of this project and then, once I get it working, I'll ping y'all.

[harrisonhjones]

@rking788 @mikeflynn PR for adding Travis CI (with instructions!): https://github.com/mikeflynn/go-alexa/pull/45

[rking788]

Looks like this can now be rebased and then merged?

[mikeflynn]

So this got kind of lost in the shuffle. Still important? If someone can resolve the conflict we can merge.