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

Moved config file to a distribution version. #16

Closed
wants to merge 4 commits into from

Conversation

scoggins
Copy link
Contributor

@scoggins scoggins commented Jan 3, 2025

  • Renamed nfc2klipper-config.json5 to nfc2klipper-config.json5.dist
  • Updated instructions in the READM.md for install
  • Added nfc2klipper-config.json5 to the .gitignore|

@bofh69 if you accept this, you'll need to copy your config back from being renamed to .dist

@bofh69
Copy link
Owner

bofh69 commented Jan 3, 2025

That's a good idea, but I started yesterday to rewrite it to use a toml file from ~/.config/nfc2config/nfc2config.cfg instead.

toml is similar to klipper and moonraker's config files and config files shouldn't really be in the script's dir.

See the switch-to-toml branch.

@scoggins scoggins closed this Jan 4, 2025
@scoggins scoggins deleted the enhancement/config_dist branch January 4, 2025 01:47
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.

2 participants