• Timothy Andrew's avatar
    Implement final review comments from @rymai. · e394d287
    Timothy Andrew authored
    - `raise "string"` raises a `RuntimeError` - no need to be explicit
    - Remove top-level comment in the `RevList` class
    - Use `%w()` instead of `%w[]`
    - Extract an `environment_variables` method to cache `env.slice(*ALLOWED_VARIABLES)`
    - Use `start_with?` for env variable validation instead of regex match
    - Validation specs for each allowed environment variable were identical. Build them dynamically.
    - Minor change to `popen3` expectation.
    e394d287
rev_list_spec.rb 1.77 KB