Skip to content

Commit 4b0155d

Browse files
author
jordanbreen28
committed
(CAT-1491) - Fix broken forge urls in Puppetfile
This commit fixes an issue where when hovering over a module in a Puppetfile, the forge url shown would be invalid. This commit aims to fix this by fixing the url, in the format now used in forge urls.
1 parent 6865447 commit 4b0155d

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/forge.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export function getPDKVersion(logger: ILogger): Promise<string> {
5757
.then((response) => {
5858
if (response.status !== 200) {
5959
logger.error(`Error getting Puppet forge data. Status: ${response.status}:${response.statusText}`);
60-
resolve();
60+
resolve(void 0);
6161
}
6262
return response.data;
6363
});
@@ -80,7 +80,7 @@ export function getModuleInfo(title: string, logger: ILogger): Promise<PuppetFor
8080
.then((response) => {
8181
if (response.status !== 200) {
8282
logger.error(`Error getting Puppet forge data. Status: ${response.status}:${response.statusText}`);
83-
resolve();
83+
resolve(void 0)
8484
}
8585

8686
const info = response.data;
@@ -93,7 +93,7 @@ export function getModuleInfo(title: string, logger: ILogger): Promise<PuppetFor
9393
created: new Date(info.created_at),
9494
updated: new Date(info.updated_at),
9595
endorsement: info.endorsement ?? '',
96-
forgeUrl: `https://forge.puppet.com/${info.owner.username}/${info.name}`,
96+
forgeUrl: `https://forge.puppet.com/modules/${info.owner.slug}/${info.name}`,
9797
homepageUrl: info.homepage_url ?? '',
9898
version: info.current_release.version,
9999
owner: {
@@ -109,7 +109,7 @@ export function getModuleInfo(title: string, logger: ILogger): Promise<PuppetFor
109109
})
110110
.catch((error) => {
111111
logger.error(`Error getting Puppet forge data: ${error}`);
112-
resolve();
112+
resolve(void 0)
113113
});
114114
});
115115
}
@@ -130,7 +130,7 @@ export function getPuppetModuleCompletion(text: string, logger: ILogger): Promis
130130
.then((response) => {
131131
if (response.status !== 200) {
132132
logger.error(`Error getting Puppet forge data. Status: ${response.status}:${response.statusText}`);
133-
resolve();
133+
resolve(void 0)
134134
}
135135

136136
const info = response.data;
@@ -144,7 +144,7 @@ export function getPuppetModuleCompletion(text: string, logger: ILogger): Promis
144144
})
145145
.catch((error) => {
146146
logger.error(`Error getting Puppet forge data: ${error}`);
147-
resolve();
147+
resolve(void 0)
148148
});
149149
});
150150
}

0 commit comments

Comments
 (0)