Adding required parameter bang notation support (will raise Configatron:... #44

Closed
wants to merge 1 commit into from

3 participants

@joehorsnell

We use configatron extensively in our Rails apps and very much like its simplicity and flexibility, but have been bitten on a few occasions by missing/misconfigured mandatory environment configuration parameters (e.g. a URL to access an external service). Rather than failing fast you get more difficult to trace runtime failures when the Configatron::Store instance returned is used as a URL. Being environment specific, this is also the kind of thing your tests don't tend to pick up.

Yes you can manually check if a given parameter is configured at runtime before using it, but this just makes it more convenient by simply appending a bang when the parameter must be there.

@joehorsnell

I have no idea why the 1.9.2 Travis CI build failed - I don't get those spec failures locally using 1.9.2

@hlascelles

Very useful change, would like to see this in configatron 3!

@markbates
Owner

An improved version of this is in 3.0.0.rc2

@markbates markbates closed this Oct 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment