Apply suggestions from code review

Co-authored-by: Robert Knight <[email protected]>
Co-authored-by: Grace <[email protected]>

Author: 3 people

src/documentation/search/search-client.ts

@@ -60,7 +60,7 @@ const workerForLanguage = (language: string) => {
   // See also convertLangToLunrParam
 
   // Enumerated for code splitting as Vite doesn't support dynamic strings here
-  // We use a worker per language for because Vite doesn't support using dynamic
+  // We use a worker per language because Vite doesn't support using dynamic
   // import in a iife Worker and Safari doesn't support module workers.
   switch (language.toLowerCase()) {
     case "de": {

src/fs/host-default.test.ts

@@ -9,7 +9,7 @@ import { defaultInitialProject } from "./initial-project";
 import { testMigrationUrl } from "./migration-test-data";
 
 describe("DefaultHost", () => {
-  it.skip("uses migration if available", async () => {
+  it("uses migration if available", async () => {
     const project = await new DefaultHost(
       testMigrationUrl
     ).createInitialProject();

src/fs/migration.test.ts

@@ -6,7 +6,7 @@
 import { isMigration, parseMigrationFromUrl } from "./migration";
 import { testMigrationUrl } from "./migration-test-data";
 
-describe.skip("parseMigrationFromUrl", () => {
+describe("parseMigrationFromUrl", () => {
   it("parses valid URL", () => {
     const migration = parseMigrationFromUrl(testMigrationUrl);
     expect(migration).toEqual({

src/language-server/pyright.ts

@@ -36,7 +36,7 @@ export const pyright = async (
     return undefined;
   }
   if (cache) {
-    // This is safe to called if already initialized.
+    // This is safe to call if already initialized.
     await cache.client.initialize();
     if (cache.language === language) {
       return cache.client;