fix: adapt for latest @octokit/rest to remove deprecation warnings (#177)

Author: immanuel192

lib/fail.js

@@ -31,7 +31,8 @@ module.exports = async (pluginConfig, context) => {
 
     if (srIssue) {
       logger.log('Found existing semantic-release issue #%d.', srIssue.number);
-      const comment = {owner, repo, number: srIssue.number, body};
+      // eslint-disable-next-line camelcase
+      const comment = {owner, repo, issue_number: srIssue.number, body};
       debug('create comment: %O', comment);
       const {
         data: {html_url: url},

lib/success.js

@@ -50,8 +50,10 @@ module.exports = async (pluginConfig, context) => {
     const prs = await pFilter(
       uniqBy(flatten(await Promise.all(searchQueries)), 'number'),
       async ({number}) =>
-        (await github.pullRequests.listCommits({owner, repo, number})).data.find(({sha}) => shas.includes(sha)) ||
-        shas.includes((await github.pullRequests.get({owner, repo, number})).data.merge_commit_sha)
+        // eslint-disable-next-line camelcase
+        (await github.pullRequests.listCommits({owner, repo, pull_number: number})).data.find(({sha}) =>
+          shas.includes(sha)
+        ) || shas.includes((await github.pullRequests.get({owner, repo, pull_number: number})).data.merge_commit_sha) // eslint-disable-line camelcase
     );
 
     debug('found pull requests: %O', prs.map(pr => pr.number));
@@ -75,7 +77,7 @@ module.exports = async (pluginConfig, context) => {
           ? template(successComment)({...context, issue})
           : getSuccessComment(issue, releaseInfos, nextRelease);
         try {
-          const comment = {owner, repo, number: issue.number, body};
+          const comment = {owner, repo, issue_number: issue.number, body};
           debug('create comment: %O', comment);
           const {
             data: {html_url: url},
@@ -118,7 +120,8 @@ module.exports = async (pluginConfig, context) => {
       srIssues.map(async issue => {
         debug('close issue: %O', issue);
         try {
-          const updateIssue = {owner, repo, number: issue.number, state: 'closed'};
+          // eslint-disable-next-line camelcase
+          const updateIssue = {owner, repo, issue_number: issue.number, state: 'closed'};
           debug('closing issue: %O', updateIssue);
           const {
             data: {html_url: url},