-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Debug information for config loading is not correct. #2152
Comments
I don't agree, we have to set defaults, it makes sense to show these defaults. The config output shows the combination of this defaults with your config. I agree there is room for improvements though, there always is :) What would the ideal output look like from your perspective? You are welcome to work on a pull request to improve it, it's not something we have the resources to do ourselves, but happy to accept a well crafted pull request |
@gr2m It's not merging. It's just showing you a default .Put this in your module.export = {
"branches": ["main"],
} Now run this,
But none of that "debug" information seems correct. The branches is not merged. Adding a |
ahh I see it now, sorry! Yes, that definitely looks like a bug. Do you want to look into it? |
@gr2m patch up. |
When I run semantic-release, I see
But actually what
release.config.js
has is this,It seems to be showing defaults, but that's not very intuitive. It's also not useful as it doesn't show the original contents at all.
The text was updated successfully, but these errors were encountered: