Skip to content

Commit 728e1eb

Browse files
authored
Use octokit's type def versions (#431)
* use octokits types * remove
1 parent 57ca50d commit 728e1eb

File tree

4 files changed

+9
-22
lines changed

4 files changed

+9
-22
lines changed

package-lock.json

-13
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
{
22
"devDependencies": {
33
"@actions/core": "1.9.0",
4-
"@octokit/openapi-types": "12.5.0",
54
"@octokit/rest": "18.12.0",
65
"@types/mocha": "9.1.1",
76
"@types/node": "16.11.41",

scripts/get-missing-cherry-picks-utils.ts

+7-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
11
import {Octokit} from '@octokit/rest';
2-
import {components} from '@octokit/openapi-types';
2+
import {GetResponseTypeFromEndpointMethod} from '@octokit/types';
3+
4+
type Commits = GetResponseTypeFromEndpointMethod<
5+
Octokit['repos']['compareCommitsWithBasehead']
6+
>['data']['commits'];
37

4-
type Commit = components['schemas']['commit'];
58
const params = {owner: 'ampproject', repo: 'amphtml'};
69

710
export async function getMissingCommits(
@@ -39,13 +42,14 @@ export async function getMissingCommits(
3942
async function getCherryPickCommits(
4043
octokit: Octokit,
4144
release: string
42-
): Promise<Commit[] | undefined> {
45+
): Promise<Commits | undefined> {
4346
if (release.endsWith('000')) return;
4447
const base = release.slice(0, -3) + '000';
4548
const response = await octokit.rest.repos.compareCommitsWithBasehead({
4649
...params,
4750
basehead: `${base}...${release}`,
4851
});
52+
4953
return response.data.commits;
5054
}
5155

scripts/promote-cherry-picks.ts

+2-5
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import {
1010
runPromoteJob,
1111
} from './promote-job';
1212
import {getChannels} from './get-channels-utils';
13-
import {components} from '@octokit/openapi-types';
1413

1514
interface Args {
1615
amp_version: string;
@@ -60,10 +59,8 @@ async function getCherryPickedPRs(
6059
if (pullNumber) {
6160
return `* https://github.com/ampproject/amphtml/pull/${pullNumber}`;
6261
}
63-
// Ugh Octokit's typing is horrendous.
64-
const {html_url: htmlUrl} =
65-
commit as unknown as components['schemas']['commit'];
66-
return `* ${htmlUrl}`;
62+
63+
return `* ${commit.url}`;
6764
});
6865
} catch (err) {
6966
console.warn('Could not fetch the list of cherry picked PRs, skipping...');

0 commit comments

Comments
 (0)