Skip to content
New issue

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

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

Already on GitHub? # to your account

Improve example config #18

Merged
merged 2 commits into from
Jul 31, 2024
Merged

Improve example config #18

merged 2 commits into from
Jul 31, 2024

Conversation

NicolasCARPi
Copy link
Contributor

Hello,

Here is a small change in the example config. It might be useful to others to see how to start it from CWD instead of a static folder.

I've also added a space after the # character, as it's visually more pleasing/readable, and generally what you'll find in most config files. As well as remove the dot at the end, but this is purely a personal preference to lighten it.

=========

  • add example for starting in current working directory
  • add a space after initial # character
  • remove trailing dots
  • change formulation for number of recently opened files kept

* add example for starting in current working directory
* add a space after initial # character
* remove trailing dots
@Bzero
Copy link
Owner

Bzero commented Jul 31, 2024

Hi,

Thanks a lot for creating this PR and for helping to make the config more easily understandable!

It might be useful to others to see how to start it from CWD instead of a static folder.

Agreed, this might be non obvious so it is a good idea to add that to the example.

I've also added a space after the # character, as it's visually more pleasing/readable, and generally what you'll find in most config files.

Yeah, that's probably more conventional.

docs/example_config.ini Outdated Show resolved Hide resolved
@Bzero Bzero merged commit f4961f9 into Bzero:master Jul 31, 2024
4 checks passed
@NicolasCARPi NicolasCARPi deleted the nico-example branch July 31, 2024 16:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants