Ensure that PRs are sorted by merge date

Author: cjbarth

lib/src/Gren.js

@@ -934,7 +934,12 @@ class Gren {
         const loaded = utils.task(this, 'Getting all merged pull requests');
         const since = releaseRanges[releaseRanges.length - 1][1].date;
         const prs = await this._getMergedPullRequests(since);
-        const filteredPrs = Array.from(prs).filter(this._filterPullRequest.bind(this));
+        prs.sort(
+            (pr1, pr2) => new Date(pr2.merged_at) - new Date(pr1.merged_at)
+        );
+        const filteredPrs = prs.filter(
+            this._filterPullRequest.bind(this)
+        );
 
         let totalPrs = 0;
         const release = releaseRanges
@@ -961,10 +966,14 @@ class Gren {
         let relevantIssues = [];
 
         const re = new RegExp('([\\w-]+)/([\\w-]+)/issues/([0-9]+)', 'gi');
+        const prs = await this._getMergedPullRequests(since);
+        prs.sort(
+            (pr1, pr2) => new Date(pr2.merged_at) - new Date(pr1.merged_at)
+        );
 
-        const prs = (await this._getMergedPullRequests(since))
+        const filteredPrs = prs
             .filter(this._filterPullRequest.bind(this))
-            .map(pr => {
+            .map((pr) => {
                 const matches = pr.body.match(re);
                 const relatedIssues = matches && matches.map(issue => {
                     const [user, repo, , number] = issue.split('/');
@@ -973,13 +982,16 @@ class Gren {
                 return Object.assign({}, pr, { relatedIssues });
             });
 
-        const release = releaseRanges
-            .map(range => {
-                const list = prs.filter(this._filterBlockPullRequest.bind(this, range));
-                list.forEach(({ relatedIssues }) => { relevantIssues = relevantIssues.concat(relatedIssues || []); });
-                loaded(`Pull Requests found: ${list.length}`);
-                return Object.assign({}, range, { list });
+        const release = releaseRanges.map((range) => {
+            const list = filteredPrs.filter(
+                this._filterBlockPullRequest.bind(this, range)
+            );
+            list.forEach(({ relatedIssues }) => {
+                relevantIssues = relevantIssues.concat(relatedIssues || []);
             });
+            loaded(`Pull Requests found: ${list.length}`);
+            return Object.assign({}, range, { list });
+        });
 
         const issuesDetails = (await Promise.all(relevantIssues.map(this._getSingleIssue.bind(this))))
             .reduce((acc, el) => { acc[el.number] = el; return acc; }, {});