Skip to content

Reorder changelog policy rules

Alessio Caiazza requested to merge nolith-master-patch-92971 into master

What does this MR do?

All the options that require a changelog entry are now listed before what does not need one.

This aims to prevent situations like !17368 (comment 221700267) where a MR with a database migration was proposed without a changelog entry because of

Any change behind a feature flag should not have a changelog entry. The entry should be added in the merge request removing the feature flags.

But several lines later another rule stated

Any change that introduces a database migration must have a changelog entry

Does this MR meet the acceptance criteria?

Conformity

Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading