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

feat(config): Use cosmiconfig defaults to support .cjs config files #1815

Conversation

opiation
Copy link
Contributor

This change adds support for *.cjs config files by removing the explicit use of searchPlaces options and relying and the default search places generated by cosmicconfig. As per the docs for cosmicconfig, the defaults include all the extensions/formats previously supported by semantic-release in addition to the new .cjs variants.

Resolves #1814.

@opiation opiation mentioned this pull request Feb 22, 2021
This change adds support for `*.cjs` config files by removing the
explicit use of `searchPlaces` options and relying and the default
search places generated by `cosmicconfig`.  As per the docs for
[`cosmicconfig`][cc], the defaults include all the extensions/formats
previously supported by semantic-release in addition to the new .cjs
variants.

Resolves [semantic-release#1814][issue].

[issue]: semantic-release#1814
[cc]: https://github.com/davidtheclark/cosmiconfig#searchplaces
@opiation opiation force-pushed the fix/support-cjs-using-cosmicconfig-defaults branch from a473f31 to 440a6b4 Compare February 26, 2021 01:16
Copy link
Member

@travi travi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good to me 👍🏼

@travi
Copy link
Member

travi commented Feb 26, 2021

thank you for investigating and tracking down these details. i did dig a little bit to try to understand why they were defined explicitly before, but even the version that resulted in the addition appears to have supported similar defaults. from what i can tell this change aligns with the original intent fro before that change too.

@travi travi changed the title Use cosmicconfig defaults to support .cjs config files feat(config): Use cosmiconfig defaults to support .cjs config files Feb 26, 2021
@travi travi merged commit 3ecc196 into semantic-release:master Feb 26, 2021
@github-actions
Copy link

🎉 This PR is included in version 17.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@opiation opiation deleted the fix/support-cjs-using-cosmicconfig-defaults branch February 26, 2021 16:35
This was referenced Mar 7, 2021
This was referenced Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support *.cjs configs
2 participants