Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

Add configuration through a settings file #22

Closed
wants to merge 5 commits into from

Conversation

Alphare
Copy link

@Alphare Alphare commented May 9, 2020

This also introduces read-only mode and a way of switching off the alternate screen as discussed in #13.

@vamolessa
Copy link
Owner

I thought #7 was refering to a cli flag (like running verco --read-only for example).
By creating a global config for read only, you want to disable writing on repositories at all on a machine?

Also, I think we can just leave the alternate screen always there, so maybe we could wait a little bit until we add a config file.

@Alphare
Copy link
Author

Alphare commented May 10, 2020

Well, I'd also like to configure whether status also shows summary (split them into two commands), because sum is much slower (for now) and I use it less frequently than status, also custom keybindings, whether to default to a command (I'd like Verco to open directly on the log), etc..
Those are just examples, but I didn't want to make a huge PR with enough to justify the settings.

I can add the cli flags if you want for --read-only/--no-read-only to overwrite the default value, but my use case is more of the "never write". I can see a setting that lets you pick which commands you want to use: I would use update, or evolve (I should make a PR), maybe histedit, but never merge, for example.

@vamolessa
Copy link
Owner

Will close this as a config file is beyond the scope of this project.

@vamolessa vamolessa closed this Apr 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants