Skip to content

Commit

Permalink
BREAKING(fs): remove Deno.File (#25447)
Browse files Browse the repository at this point in the history
Towards #22079
  • Loading branch information
iuioiua authored Sep 5, 2024
1 parent 0450c12 commit 713ed06
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 21 deletions.
11 changes: 0 additions & 11 deletions cli/tsc/dts/lib.deno.ns.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2530,17 +2530,6 @@ declare namespace Deno {
[Symbol.dispose](): void;
}

/**
* The Deno abstraction for reading and writing files.
*
* @deprecated This will be removed in Deno 2.0. See the
* {@link https://docs.deno.com/runtime/manual/advanced/migrate_deprecations | Deno 1.x to 2.x Migration Guide}
* for migration instructions.
*
* @category File System
*/
export const File: typeof FsFile;

/** Gets the size of the console as columns/rows.
*
* ```ts
Expand Down
3 changes: 0 additions & 3 deletions ext/fs/30_fs.js
Original file line number Diff line number Diff line change
Expand Up @@ -786,8 +786,6 @@ function checkOpenOptions(options) {
}
}

const File = FsFile;

function readFileSync(path) {
return op_fs_read_file_sync(pathFromURL(path));
}
Expand Down Expand Up @@ -950,7 +948,6 @@ export {
cwd,
fdatasync,
fdatasyncSync,
File,
FsFile,
fsync,
fsyncSync,
Expand Down
2 changes: 0 additions & 2 deletions runtime/js/99_main.js
Original file line number Diff line number Diff line change
Expand Up @@ -801,7 +801,6 @@ function bootstrapMainRuntime(runtimeOptions, warmup = false) {
if (internals.future) {
delete globalThis.window;
delete Deno.Buffer;
delete Deno.File;
delete Deno.FsFile.prototype.rid;
delete Deno.funlock;
delete Deno.funlockSync;
Expand Down Expand Up @@ -965,7 +964,6 @@ function bootstrapWorkerRuntime(

if (internals.future) {
delete Deno.Buffer;
delete Deno.File;
delete Deno.FsFile.prototype.rid;
delete Deno.funlock;
delete Deno.funlockSync;
Expand Down
1 change: 0 additions & 1 deletion tests/specs/future/runtime_api/main.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
console.log("window is", globalThis.window);
console.log("Deno.Buffer is", Deno.Buffer);
console.log("Deno.File is", Deno.File);
console.log(
"Deno.FsFile.prototype.rid is",
Deno.openSync(import.meta.filename).rid,
Expand Down
1 change: 0 additions & 1 deletion tests/specs/future/runtime_api/main.out
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
window is undefined
Deno.Buffer is undefined
Deno.File is undefined
Deno.FsFile.prototype.rid is undefined
Deno.funlock is undefined
Deno.funlockSync is undefined
Expand Down
3 changes: 0 additions & 3 deletions tests/unit/files_test.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
// Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.

// deno-lint-ignore-file no-deprecated-deno-api

import {
assert,
assertEquals,
Expand All @@ -24,7 +22,6 @@ Deno.test(
async function filesCopyToStdout() {
const filename = "tests/testdata/assets/fixture.json";
using file = await Deno.open(filename);
assert(file instanceof Deno.File);
assert(file instanceof Deno.FsFile);
assert(file.rid > 2);
const bytesWritten = await copy(file, Deno.stdout);
Expand Down

0 comments on commit 713ed06

Please sign in to comment.