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

Use YetAnotherConfigLib rather than CompleteConfig #97

Open
wants to merge 3 commits into
base: 1.19
Choose a base branch
from

Conversation

isXander
Copy link

@isXander isXander commented Nov 27, 2022

Benefits include:

Disadvantages include:

  • requires the library, you can't load CEM without it.

@offbeat-stuff
Copy link

cloth config is more used, so I think he should not be using something else, its stable and so its just better to use a standard config lib

@dorianpb
Copy link
Owner

well I could just include it

@dorianpb
Copy link
Owner

I like the syntax more from a first glance, I'll test it and see if I like it more

@dorianpb
Copy link
Owner

there also shouldn't a firsr-time crash with complete config, that's a separate issue

@isXander
Copy link
Author

cloth config is more used, so I think he should not be using something else, its stable and so its just better to use a standard config lib

Shedaniel doesn't recommend usage of cloth config.

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.

3 participants