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

Allow configuration file to be extended #425

Closed
blephy opened this issue Nov 4, 2022 · 4 comments
Closed

Allow configuration file to be extended #425

blephy opened this issue Nov 4, 2022 · 4 comments
Labels
feature-request help wanted Issue is well defined but needs assignment

Comments

@blephy
Copy link

blephy commented Nov 4, 2022

I,

It would be great to be able to extends a configuration file to another, like TSConfig or Eslint do, or at least to be able to write .cjs / .js configuration file to require another js file.

We use c8 in a mono-repo, and we cannot share a c8 configuration file between workspace:/

Thanks

@bcoe bcoe added help wanted Issue is well defined but needs assignment feature-request labels Nov 11, 2022
@bcoe
Copy link
Owner

bcoe commented Nov 11, 2022

This seems like a good feature request 👍

@blephy
Copy link
Author

blephy commented Nov 12, 2022

Do you need some help ?

@blephy
Copy link
Author

blephy commented Nov 20, 2022

@bcoe i did look to the c8 and yargs code. In fact, we can already extends configuration if the path to the extendable config is a json file.

This is already working :

{
  "extends": "../../config/c8/c8.config.json",
  // ...
}

What about making c8 compatible with yml and js configuration ? cosmiconfig sounds good for this.

Additionally, we should update the documentation. I did a branch but cannot push to this repository :/

@bcoe
Copy link
Owner

bcoe commented Nov 21, 2022

Closing in favour of:

#431

With regards to cosmicconfig, I would prefer not to pull in a dependency for config management, if it can be avoided. I think it's not worth the complexity it would add to the library.

Additionally, we should update the documentation. I did a branch but cannot push to this repository :/

You will need to create a pull request form a fork, here's some documentation on how to do so 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request help wanted Issue is well defined but needs assignment
Projects
None yet
Development

No branches or pull requests

2 participants