Skip to content

Commit fb8bf97

Browse files
committed
Remove obsolete warning (microsoft#170049)
1 parent 7a7d457 commit fb8bf97

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

extensions/extension-editing/src/extensionLinter.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ const svgsNotValid = l10n.t("SVGs are not a valid image source.");
3030
const embeddedSvgsNotValid = l10n.t("Embedded SVGs are not a valid image source.");
3131
const dataUrlsNotValid = l10n.t("Data URLs are not a valid image source.");
3232
const relativeUrlRequiresHttpsRepository = l10n.t("Relative image URLs require a repository with HTTPS protocol to be specified in the package.json.");
33-
const relativeIconUrlRequiresHttpsRepository = l10n.t("An icon requires a repository with HTTPS protocol to be specified in this package.json.");
3433
const relativeBadgeUrlRequiresHttpsRepository = l10n.t("Relative badge URLs require a repository with HTTPS protocol to be specified in this package.json.");
3534
const apiProposalNotListed = l10n.t("This proposal cannot be used because for this extension the product defines a fixed set of API proposals. You can test your extension but before publishing you MUST reach out to the VS Code team.");
3635
const implicitActivationEvent = l10n.t("This activation event cannot be explicitly listed by your extension.");
@@ -458,11 +457,10 @@ export class ExtensionLinter {
458457
diagnostics.push(new Diagnostic(range, dataUrlsNotValid, DiagnosticSeverity.Warning));
459458
}
460459

461-
if (!hasScheme && !info.hasHttpsRepository) {
460+
if (!hasScheme && !info.hasHttpsRepository && context !== Context.ICON) {
462461
const range = new Range(document.positionAt(begin), document.positionAt(end));
463462
const message = (() => {
464463
switch (context) {
465-
case Context.ICON: return relativeIconUrlRequiresHttpsRepository;
466464
case Context.BADGE: return relativeBadgeUrlRequiresHttpsRepository;
467465
default: return relativeUrlRequiresHttpsRepository;
468466
}

0 commit comments

Comments
 (0)