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

[FR] Introduce a config file #23

Open
B00tLoad opened this issue Feb 2, 2023 · 1 comment
Open

[FR] Introduce a config file #23

B00tLoad opened this issue Feb 2, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@B00tLoad
Copy link
Owner

B00tLoad commented Feb 2, 2023

Is your feature request related to a problem? Please describe.
Supplying configuration via the command line is a bit clunky.

Describe the solution you'd like
A config file to set all values

@B00tLoad B00tLoad added the enhancement New feature or request label Feb 2, 2023
@B00tLoad B00tLoad changed the title [FR] Introdice a config file [FR] Introduce a config file Feb 2, 2023
@B00tLoad B00tLoad assigned B00tLoad and unassigned B00tLoad Jun 1, 2023
@B00tLoad
Copy link
Owner Author

B00tLoad commented Jun 1, 2023

When implementing please have a look at the config setup over at https://github.com/B00tLoad/SpotifyDedupe and try to make it as close to that as possible.

Musts for merging a pull request:

  • use TOML for the config file
  • save it at "~/.bsu/lfm2s"
  • implement a default config file, and check if it has been changed before running the tool
  • should self-fix on version change

Feel free to port the setup from the dedupe tool over to this, I don't have the time for it right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant