-
Notifications
You must be signed in to change notification settings - Fork 10
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
handle release_summary from config #16
Conversation
@@ -125,6 +125,8 @@ def is_valid_changelog_format(path: str) -> bool: | |||
changelog_config = yaml.safe_load(config_file) | |||
changes_type = tuple(item[0] for item in changelog_config["sections"]) | |||
changes_type += (changelog_config["trivial_section_name"],) | |||
changes_type += (changelog_config["prelude_section_name"],) | |||
changes_type += (changelog_config["new_plugins_after_name"],) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, I mean I kinda get what it's doing, I just don't understand the key name here. Might be beyond the scope of this PR though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is correct, I am not sure what section should be prelude_section_name
I'll take it out!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you take prelude section name out, it wouldn't recognize release_summary
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NilashishC My bad, removed new_plugins_after_name
now, confused it with prelude_section_name
while removing did not check config.yml 😿
No description provided.