Skip to content

Commit b0e9648

Browse files
committed
Addressing code review comments
1 parent 2607eec commit b0e9648

File tree

1 file changed

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

1 file changed

+2
-4
lines changed

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

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,21 +65,19 @@ class ManifestFetcher {
6565
}
6666

6767
private fun fetchManifestFromLocal(): ManifestManager.Manifest? {
68-
var manifest: ManifestManager.Manifest? = null
6968
val localETag = getManifestETagFromLocal()
7069
val remoteETag = getManifestETagFromUrl()
7170
// If local and remote have same ETag, we can re-use the manifest file from local to fetch artifacts.
7271
if (localETag != null && remoteETag != null && localETag == remoteETag) {
7372
try {
7473
val manifestContent = lspManifestFilePath.readText()
75-
manifest = manifestManager.readManifestFile(manifestContent) ?: return null
74+
return manifestManager.readManifestFile(manifestContent)
7675
} catch (e: Exception) {
7776
logger.error("error reading lsp manifest file from local ${e.message}", e)
7877
return null
7978
}
8079
}
81-
logger.info("Re-using lsp manifest from local.")
82-
return manifest
80+
return null
8381
}
8482

8583
private fun getManifestETagFromLocal(): String? {

0 commit comments

Comments
 (0)