sort categories from new features, bugfixes and removed

Author: TomasVotruba

Diff for: src/ChangelogContentsFactory.php

@@ -43,7 +43,13 @@ public function create(array $changelogLines): string
         Assert::allString($changelogLines);
 
         // summarize into "Added Features" and "Bugfixes" groups
-        $linesByCategory = [];
+        $linesByCategory = [
+            // set order clearly here
+            ChangelogCategory::NEW_FEATURES => [],
+            ChangelogCategory::BUGFIXES => [],
+            ChangelogCategory::REMOVED => [],
+            ChangelogCategory::SKIPPED => [],
+        ];
 
         foreach ($changelogLines as $changelogLine) {
             foreach (self::FILTER_KEYWORDS_BY_CATEGORY as $category => $filterKeywords) {
@@ -62,6 +68,9 @@ public function create(array $changelogLines): string
         // remove skipped lines
         unset($linesByCategory[ChangelogCategory::SKIPPED]);
 
+        // remove empty categories
+        $linesByCategory = array_filter($linesByCategory);
+
         return $this->generateFileContentsFromGroupedItems($linesByCategory);
     }
 

Diff for: tests/ChangelogContentsFactory/ChangelogContentsFactoryTest.php

@@ -19,9 +19,10 @@ protected function setUp(): void
     public function test(): void
     {
         $changelogLines = [
-            '* Add new rule',
             '* Fix bug',
+            '* Add new rule',
             '* Fixed another bug',
+            '* Removed old rule',
             '* Enable PHPStan on tests as well + add "unused public" ([#3238](https://github.com/rectorphp/rector-src/pull/3238))',
         ];
 

Diff for: tests/ChangelogContentsFactory/Fixture/generated_changelog.md

@@ -8,3 +8,9 @@
 
 * Fix bug
 * Fixed another bug
+
+<br>
+
+## Removed :skull:
+
+* Removed old rule