Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds ability to configure default config path #54

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rossta
Copy link
Contributor

@rossta rossta commented Nov 22, 2024

The Litestream Ruby config path defaults to "config/litestream.yml". Some applications may wish to use a custom configuration path, say, one that is based on the Rails environment "config/litestream/#{Rails.env}.yml"

Though Litestream::Commands methods allow for passing a --config option for the config path, helper methods like Litestream.verify!, which wraps Litestream::Commands.databases and the Puma plugin, which wraps Litestream::Commands.replicate, do not.

This changeset adds the ability to configure an application-wide default for the litestream config path allows for use of features like Litestream.verify! and the Puma plugin without otherwise having to modify these features to accept the --config option.

Though Litestream::Commands methods allow for passing a --config option
for the config path, helper methods like Litestream.verify!, which wraps
Litestream::Commands.databases and the Puma plugin, which wraps
Litestream::Commands.replicate, do not.

This changeset adds the ability to configure an application-wide default
for the litestream config path allows for use of features like
Litestream.verify! and the Puma plugin without otherwise having to
modify these features to accept the --config option.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant