Skip to content

Commit 8a43b85

Browse files
committed
refactor: enhance buildReleaseArtifacts
1 parent 199c3e6 commit 8a43b85

File tree

3 files changed

+6
-13
lines changed

3 files changed

+6
-13
lines changed

apps/site/components/withDownloadArchive.tsx

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import {
1111
type DownloadArchive = ReturnType<typeof buildReleaseArtifacts>;
1212

1313
type WithDownloadArchiveProps = {
14-
children: FC<DownloadArchive>;
14+
children: FC<DownloadArchive & { majorVersions: Array<string> }>;
1515
};
1616

1717
/**
@@ -37,13 +37,9 @@ const WithDownloadArchive: FC<WithDownloadArchiveProps> = async ({
3737
}
3838

3939
const majorVersions = releaseData.map(release => release.versionWithPrefix);
40-
const releaseArtifacts = buildReleaseArtifacts(
41-
release,
42-
version,
43-
majorVersions
44-
);
40+
const releaseArtifacts = buildReleaseArtifacts(release, version);
4541

46-
return <Component {...releaseArtifacts} />;
42+
return <Component {...releaseArtifacts} majorVersions={majorVersions} />;
4743
};
4844

4945
export default WithDownloadArchive;

apps/site/pages/en/download/archive.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ layout: download-archive
44
---
55

66
<WithDownloadArchive>
7-
{({ binaries, installers, version, release, sources, majors }) => (
7+
{({ binaries, installers, version, release, sources, majorVersions }) => (
88
<>
99
<h1>Download Node.js® {version}</h1>
1010
<ReleaseOverview release={release} />
@@ -41,7 +41,7 @@ layout: download-archive
4141
<h2 className="inline">Other releases</h2>
4242
</summary>
4343
<ul>
44-
{majors.map(version => (
44+
{majorVersions.map(version => (
4545
<li key={version}>
4646
<Link href={`/download/${version}`}>
4747
<h3>Node.js {version}</h3>

apps/site/util/downloadUtils/archive.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,7 @@ const getCompatibleArtifacts = ({
9090

9191
export const buildReleaseArtifacts = (
9292
release: NodeRelease,
93-
version: string,
94-
majors: Array<string>
93+
version: string
9594
) => {
9695
const minorVersion = release.minorVersions.find(
9796
({ versionWithPrefix }) => versionWithPrefix === version
@@ -123,9 +122,7 @@ export const buildReleaseArtifacts = (
123122
}),
124123
},
125124
version: version,
126-
minors: enrichedRelease.minorVersions,
127125
release: enrichedRelease,
128-
majors: majors,
129126
};
130127
};
131128

0 commit comments

Comments
 (0)