Skip to content

Commit c557a7f

Browse files
Three minor clarifications (#25621)
## Description Fix three minor code readability issues.
1 parent 93fe943 commit c557a7f

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/drivers/local-driver/src/localCreateDocument.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ export async function createDocument(
1717
localDeltaConnectionServer,
1818
resolvedUrl: IResolvedUrl,
1919
summary: ISummaryTree,
20-
) {
20+
): Promise<void> {
2121
const pathName = new URL(resolvedUrl.url).pathname;
2222
const pathArr = pathName.split("/");
2323
// TODO Why are we non null asserting here

packages/test/test-utils/src/loaderContainerTracker.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import { canBeCoalescedByService } from "@fluidframework/driver-utils/internal";
2525
import { toIDeltaManagerFull, waitForContainerConnection } from "./containerUtils.js";
2626
import { debug } from "./debug.js";
2727
import { isNonEmptyArray, type NonEmptyArray } from "./nonEmptyArrayType.js";
28-
import { IOpProcessingController } from "./testObjectProvider.js";
28+
import type { IOpProcessingController } from "./testObjectProvider.js";
2929
import { timeoutAwait, timeoutPromise } from "./timeoutUtils.js";
3030

3131
const debugOp = debug.extend("ops");
@@ -240,7 +240,7 @@ export class LoaderContainerTracker implements IOpProcessingController {
240240
break;
241241
}
242242

243-
// Ignore readonly dirty containers, because it can't sent ops and nothing can be done about it being dirty
243+
// Ignore readonly dirty containers because they can't send ops and nothing can be done about them being dirty.
244244
const dirtyContainers = containersToApply.filter((c) => {
245245
const { deltaManager, isDirty } = c;
246246
return deltaManager.readOnlyInfo.readonly !== true && isDirty;

0 commit comments

Comments
 (0)