Skip to content

Commit 8f44478

Browse files
committed
Fixing detektMain lint issues
1 parent 563ca08 commit 8f44478

File tree

1 file changed

+8
-6
lines changed
  • plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts

1 file changed

+8
-6
lines changed

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/artifacts/ManifestFetcher.kt

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,10 @@ package software.aws.toolkits.jetbrains.services.amazonq.lsp.artifacts
55

66
import com.intellij.openapi.util.text.StringUtil
77
import com.intellij.util.io.DigestUtil
8+
import software.aws.toolkits.core.utils.error
89
import software.aws.toolkits.core.utils.exists
910
import software.aws.toolkits.core.utils.getLogger
11+
import software.aws.toolkits.core.utils.info
1012
import software.aws.toolkits.core.utils.readText
1113
import software.aws.toolkits.jetbrains.core.getETagFromUrl
1214
import software.aws.toolkits.jetbrains.core.getTextFromUrl
@@ -42,23 +44,23 @@ class ManifestFetcher {
4244
val manifestString = getTextFromUrl(lspManifestUrl)
4345
manifest = manifestManager.readManifestFile(manifestString) ?: return null
4446
} catch (e: Exception) {
45-
logger.error("error fetching lsp manifest from remote URL ${e.message}", e)
47+
logger.error(e) { "error fetching lsp manifest from remote URL ${e.message}" }
4648
return null
4749
}
4850
if (manifest.isManifestDeprecated == true) {
49-
logger.info("Manifest is deprecated")
51+
logger.info { "Manifest is deprecated" }
5052
return null
5153
}
5254
updateManifestCache()
53-
logger.info("Using manifest found from remote URL")
55+
logger.info { "Using manifest found from remote URL" }
5456
return manifest
5557
}
5658

5759
private fun updateManifestCache() {
5860
try {
5961
saveFileFromUrl(lspManifestUrl, lspManifestFilePath)
6062
} catch (e: Exception) {
61-
logger.error("error occurred while saving lsp manifest to local cache ${e.message}", e)
63+
logger.error(e) { "error occurred while saving lsp manifest to local cache ${e.message}" }
6264
}
6365
}
6466

@@ -71,7 +73,7 @@ class ManifestFetcher {
7173
val manifestContent = lspManifestFilePath.readText()
7274
return manifestManager.readManifestFile(manifestContent)
7375
} catch (e: Exception) {
74-
logger.error("error reading lsp manifest file from local ${e.message}", e)
76+
logger.error(e) { "error reading lsp manifest file from local ${e.message}" }
7577
return null
7678
}
7779
}
@@ -92,7 +94,7 @@ class ManifestFetcher {
9294
val actualETag = getETagFromUrl(lspManifestUrl)
9395
return actualETag.trim('"')
9496
} catch (e: Exception) {
95-
logger.error("error fetching ETag of lsp manifest from url.", e)
97+
logger.error(e) { "error fetching ETag of lsp manifest from url." }
9698
}
9799
return null
98100
}

0 commit comments

Comments
 (0)