Skip to content

Commit

Permalink
🐝 fix minor issues
Browse files Browse the repository at this point in the history
  • Loading branch information
danyx23 committed Aug 7, 2024
1 parent 24eeed8 commit 7c77d87
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
1 change: 0 additions & 1 deletion adminSiteServer/chartConfigR2Helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ import {
Base64String,
excludeUndefined,
JsonError,
getMd5HashBase64,
} from "@ourworldindata/utils"
import { logErrorAndMaybeSendToBugsnag } from "../serverUtils/errorLog.js"
import { createHash } from "crypto"
Expand Down
5 changes: 3 additions & 2 deletions devTools/syncGraphersToR2/syncGraphersToR2.ts
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ async function syncWithR2(
const full = fullConfigMap.get(id)
if (full === undefined) {
console.error(`Full config not found for id ${id}`)
return
return null
}
try {
const putObjectCommandInput: PutObjectCommandInput = {
Expand All @@ -148,10 +148,11 @@ async function syncWithR2(
new PutObjectCommand(putObjectCommandInput)
)
else console.log("Would have upserted", key)
progressBar.tick()
return null
} catch (err) {
return err
}
progressBar.tick()
})
const promiseResults = await Promise.allSettled(uploadPromises)
const batchErrors = promiseResults
Expand Down

0 comments on commit 7c77d87

Please sign in to comment.