Skip to content
This repository has been archived by the owner on Jun 8, 2024. It is now read-only.

fix(global config): renamed default-config to typstfmt #118

Merged

Conversation

Andrew15-5
Copy link
Contributor

It is better to name global config the same way as the local config. This way any local config can quickly become a global config (without renaming the file).

It is better to name global config the same way as the local config.
This way any local config can quickly become a global config (without
renaming the file).
src/main.rs Outdated Show resolved Hide resolved
@astrale-sharp
Copy link
Owner

Note: We could add an option to let the user provide a config path for testing purposes (but we don't have to, let's wait the need for it first)

Merged `CONFY_APP_NAME` and `GLOBAL_CONFIG_BASE_NAME` into `APP_NAME`.
It is shorter and more meaningful.
@astrale-sharp
Copy link
Owner

With your latest change, the file ends up in /home/astrale/.config/typstfmt/typstfmt.toml which is perfect ;)
ready to merge when you are

@Andrew15-5
Copy link
Contributor Author

With your latest change, the file ends up in /home/astrale/.config/typstfmt/typstfmt.toml which is perfect ;)

Well, yes, that was the idea.

ready to merge when you are

I was born ready.

@astrale-sharp astrale-sharp merged commit 029c719 into astrale-sharp:master Oct 31, 2023
1 check passed
@astrale-sharp
Copy link
Owner

Thanks!

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