Update CHANGELOG.md and apply lints

Author: brizental

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 [Full changelog](https://github.com/mozilla/glean.js/compare/v0.17.0...main)
 
-* [#542](https://github.com/mozilla/glean.js/pull/542): Implement `shutdown` API.
+* [#542](https://github.com/mozilla/glean.js/pull/542), [#552](https://github.com/mozilla/glean.js/pull/552): Implement `shutdown` API.
 
 # v0.17.0 (2021-07-16)
 

glean/src/core/upload/index.ts

@@ -129,9 +129,8 @@ class PingUploader implements PingsDatabaseObserver {
 
     // If the ping is a deletion-request ping, we want to enqueue it as a persistent task,
     // so that clearing the queue does not clear it.
-    let launchFn = isDeletionRequest(ping)
-      ? this.dispatcher.launchPersistent
-        : this.dispatcher.launch;
+    // eslint-disable-next-line @typescript-eslint/unbound-method
+    const launchFn = isDeletionRequest(ping) ? this.dispatcher.launchPersistent : this.dispatcher.launch;
 
     // Dispatch the uploading task.
     launchFn.bind(this.dispatcher)(async (): Promise<void> => {
@@ -238,7 +237,14 @@ class PingUploader implements PingsDatabaseObserver {
         finalPing.headers
       );
     } catch(e) {
-      log(LOG_TAG, ["Error trying to build ping request:", e.message], LoggingLevel.Warn);
+      log(
+        LOG_TAG,
+        [
+          "Error trying to build ping request:",
+          (e as Error).message
+        ],
+        LoggingLevel.Warn
+      );
       // An unrecoverable failure will make sure the offending ping is removed from the queue and
       // deleted from the database, which is what we want here.
       return {

glean/tests/unit/core/upload/index.spec.ts

@@ -75,7 +75,7 @@ describe("PingUploader", function() {
 
   it("clearing does not clear deletion-request ping job", async function () {
     const httpClient = new WaitableUploader();
-    const postSpy = sandbox.spy(httpClient, "post")
+    const postSpy = sandbox.spy(httpClient, "post");
     await Glean.testResetGlean(testAppId, true, { httpClient });