veracross / consult

Configuration and secrets management for Rails
MIT License
12 stars 0 forks source link

Consider removing the dependency on ActiveSupport #26

Closed BrianBorge closed 4 years ago

BrianBorge commented 4 years ago

Consider removing the dependency on ActiveSupport.

The con here is that Consult would lose access to the set of utility methods ActiveSupport provides, which are nice but Consult makes use of only 3 of these methods.

Removing ActiveSupport would mean one less dependency to manage and a smaller deployment package for projects that take a dependency on Consult.

Consult uses the following ActiveSupport methods:

Worth ripping out?

BrianBorge commented 4 years ago

Addressed by https://github.com/veracross/consult/pull/27