I think it would be cleaner to use the `merge()` function here as opposed to setting several instance variables to hold this data. The pattern that I'm using here in other modules that I was planning on using in this module is:

```
class nginx::config(
  $options = {},
) {
  $config = merge($github::params::defaults, $options)
  ...

}
```

and then use the hash to reference various variables in the hash where needed in the manifest itself.