Axetroy axetroy NanNing, China axetroy.xyz 说!!!为什么关注我?

push eventfinished-product/company_website

axetroy

commit sha 8a8838ffd3108c08df5a4cfa05947842a636bd9d

update

view details

push time in 7 minutes

push eventfinished-product/company_website

axetroy

commit sha b5226bfb2d781c24a82c446c53daab7cfabe7ba7

update

view details

push time in 28 minutes

fork axetroy/company_website

node.js+layui+bootstrap快速开发企业网站

fork in 2 hours

push eventaxetroy/deno_std

Axetroy

commit sha 59adafe86710038c6864fc0686f0292bf7fd9b1d

fix: wrong usage of assertThrowsAsync which without await keyword (#295)

view details

Axetroy

commit sha a00c51b05b6172d250b04a3bb6c244e5276c2704

remove unnecessary path.resolve in move/readJson/writeJson (#292)

view details

Axetroy

commit sha e80f14489084d748bb52c4311519f5d0c7696f47

move test file out of tests dir in flags module (#293)

view details

Axetroy

commit sha cf7a79f374547f03e9522723cc73d285473910fc

Merge branch 'master' into ensure

view details

push time in 2 hours

delete branch axetroy/deno_std

delete branch : flags

delete time in 2 hours

issue commentdawnlabs/carbon

Feature Request: Carbon as an API

One year has passed

Is there any plan for this API? Is there anyone to implement it?

deadcoder0904

comment created time in 20 hours

fork axetroy/nsfwjs

NSFW detection on the client-side via Tensorflow JS

https://nsfwjs.com/

fork in 21 hours

push eventaxetroy/deno_std

axetroy

commit sha cd9abdb8bd55c9a73af3534e72b5189aae4e5e24

fix assertThrowsAsync without await

view details

push time in 21 hours

push eventaxetroy/deno_std

axetroy

commit sha c78607c0258ba00f0497cf034bfd74bd48ca8166

add test case for utils getFileInfoType

view details

push time in a day

create barnchaxetroy/deno_std

branch : async_test

created branch time in a day

push eventaxetroy/deno_std

axetroy

commit sha 9aff72185f739e202c6fd4cd9ea77b345e10fbb5

fix test

view details

push time in a day

push eventaxetroy/deno_std

axetroy

commit sha ededbedd69173b1f6a1142cbde28f9904daa138b

update comment

view details

push time in a day

create barnchaxetroy/deno_std

branch : ensure

created branch time in a day

PR opened denoland/deno_std

move test file out of tests dir in flags module
+52 -52

0 comment

14 changed files

pr created time in a day

create barnchaxetroy/deno_std

branch : flags

created branch time in a day

push eventaxetroy/deno_std

axetroy

commit sha c14e0a25971ea9f1d3a23e763143a163bd746523

remove unnecessary path.resolve in move/readJson/writeJson

view details

push time in a day

create barnchaxetroy/deno_std

branch : test_rename

created branch time in a day

push eventaxetroy/deno_std

axetroy

commit sha 6c064fbbcee2dad346b47964fa41faed29b8c510

update comment

view details

push time in a day

PR opened denoland/deno_std

fix: fs.exists not work for symlink
+11 -2

0 comment

3 changed files

pr created time in a day

create barnchaxetroy/deno_std

branch : exists

created branch time in a day

push eventaxetroy/go-server

axetroy

commit sha bf7251d117920974f4caa8c56cd5055638e12010

add main.go

view details

push time in a day

push eventaxetroy/go-server

axetroy

commit sha 31c9ec5a743bb7a514b219b6fcf3033e10830f50

add main.go

view details

push time in a day

push eventaxetroy/go-server

axetroy

commit sha def4f4e7e7b1fe3d03b24ca00fc274730c92b362

update

view details

push time in a day

push eventaxetroy/go-server

axetroy

commit sha a4af5eeb6b119d0315b0e3d41056966933a2ac0c

feat: 支持google oauth2认证登陆

view details

push time in 2 days

pull request commentdenoland/deno_std

feat: add assertNotThrows/assertNotThrowsAsync for testing/assert module

Yes, I think about it again. It really is not suitable

axetroy

comment created time in 2 days

pull request commentdenoland/deno_std

feat: add assertNotThrows/assertNotThrowsAsync for testing/assert module

@zekth

Because I have write the test case with assertThrows/assertThrowsAsync.

In some scenarios, in order to implement assertNotThrowsAsync

I have to write like this:

https://github.com/denoland/deno_std/blob/fae48d5dfbbe6267c71a0dc48647548b190dd875/fs/write_json_test.ts#L35-L46

I can do this more simply and clear with assertNotThrowsAsync.

await Deno.writeFile(existsJsonFile, new Uint8Array());

await assertNotThrowsAsync(() => writeJson(existsJsonFile, { a: '1' }));

const content = await Deno.readFile(existsJsonFile);
axetroy

comment created time in 2 days

create barnchaxetroy/deno_std

branch : asserts_not_throws

created branch time in 2 days

PR closed denoland/deno_std

[WIP] feat: add http client module

ref #284

If you think that this should be what std package should do, then I will continue to complete the rest.

or it should be done by a third-party library?

how do you think

+111 -0

5 comments

1 changed file

axetroy

pr closed time in 2 days

pull request commentdenoland/deno_std

[WIP] feat: add http client module

I think so too.

just asking for your opinions.

End the discussion.

axetroy

comment created time in 2 days

issue openeddenoland/deno

Feature Request: Deno.utimes() api

Deno.utimes() change the file system timestamps of the object referenced by path.

created time in 2 days

pull request commentdenoland/deno_std

feat: add copy/copySync for fs modules

Here's a question:

  1. Deno does not have API to update files'creation time now.
  2. Deno does not have an API to delete the link file https://github.com/denoland/deno/issues/1947
axetroy

comment created time in 2 days

issue openeddenoland/deno

Deno.remove() can not remove symlink file

I found Deno.remove() cannot remove symlink file

This should be abnormal

Or we should need a Deno.unlink()?

created time in 2 days

delete branch axetroy/deno_std

delete branch : is_sub_dir

delete time in 2 days

push eventaxetroy/deno_std

axetroy

commit sha a5231ca22c8d44fb611e5a0fa80179f8ea63700b

fix: import without ext

view details

push time in 2 days

pull request commentdenoland/deno_std

[WIP] feat: add http client module

@ry Fetch-based encapsulation provides a faster way to send an HTTP request.

Maybe people think that the interface of the axios class is better than fetch.

So try to integrate into std

axetroy

comment created time in 3 days

pull request commentdenoland/deno_std

extract internal method isSubdir to fs/utils.ts

@ry yes. move to fs/utils.ts

axetroy

comment created time in 3 days

push eventaxetroy/deno_std

axetroy

commit sha c7b686628fdf98e9fc6c3cbf22900bcad056345d

fix eslint

view details

push time in 3 days

push eventaxetroy/deno_std

axetroy

commit sha a5ff41adec47705c5fb88a99e85e507a31c669f0

refactor: move to fs/utils.ts

view details

push time in 3 days

startedinfinitered/nsfwjs

started time in 3 days

PR opened denoland/deno_std

[WIP] feat: add http client module

ref #284

If you think that this should be what std package should do, then I will continue to complete the rest.

or it should be done by a third-party library?

how do you think

+111 -0

0 comment

1 changed file

pr created time in 3 days

create barnchaxetroy/deno_std

branch : http_client

created branch time in 3 days

push eventaxetroy/deno_std

axetroy

commit sha 57af76b84f038f4874a408fb59480825449952d4

use unimplemented() instead of throw by it self

view details

push time in 3 days

issue commentdenoland/deno_std

lodash like functions

agree with @bartlomieju.

we should not reinvent the wheel.

Lodash is a great library, but it can still run in deno

zekth

comment created time in 3 days

issue commentdenoland/deno_std

Axios-like wrapper on top of fetch

axios base on XMLHTTP in browser. but deno does not support it.

it means some methods like onUploadProgress and onDownloadProgress can not be implemented.

If we need to make an HTTP library like axios. Then it should not be called axios

Maybe it should be the sub-module of http

import * as http from 'https://deno.land/x/std/http/client/mod.ts';

const instance = http.create({ base: 'http://localhost:8080' })

http.get('/')
instance.get('/')
...
sh7dm

comment created time in 3 days

push eventaxetroy/deno_std

axetroy

commit sha 309cba5f063e0d39b11d9ac90c4a045f5298ade4

update error message for test

view details

push time in 3 days

push eventaxetroy/deno_std

axetroy

commit sha 81bc6a60e360cf82c1d509c2626e0cc7780aba5e

fix in windows

view details

push time in 3 days

PR opened denoland/deno_std

extract internal method isSubdir to path modules
+61 -11

0 comment

4 changed files

pr created time in 3 days

create barnchaxetroy/deno_std

branch : is_sub_dir

created branch time in 3 days

issue closedaxetroy/todo

写一个vscode自动补全注释的扩展

初衷/目标

<!-- 要做这个的目的 -->

因为我老是记不住一些注释,比如 eslint 忽略下一行等

TODO 类型

  • [x] 类库
  • [ ] 框架
  • [ ] 服务
  • [ ] 命令行工具

预计时间

<!-- 计划要完成的时间 -->

  • [ ] 一天
  • [x] 一周
  • [ ] 两周
  • [ ] 一月
  • [ ] 更久

相关链接

https://github.com/axetroy/vscode-comment-autocomplete

closed time in 4 days

axetroy

startedmozilla/send

started time in 4 days

issue commentaxetroy/vscode-comment-autocomplete

[User experience] Invoke completion when adding comments using shortcut keys.

按下command + /的时候就是执行一个命令,跟智能提示没有关系

vscode 并没有提供键盘的输入监听

就得做监听文档的变更,然后判断当前光标的前面字符串是不是注释标识符, 比如这里的 //,然后触发智能提示

有点麻烦,而且也费性能,所以这个暂时不会考虑,除非又新的 API 或者更好的方案

LeuisKen

comment created time in 5 days

fork axetroy/shadowsocks-heroku

shadowsocks over WebSocket, support Heroku.

fork in 5 days

fork axetroy/now-github-starter

Starter project to demonstrate a project whose pull requests get automatically deployed

fork in 5 days

issue closeddenoland/deno_std

refactor fs to following style guide

here to remind me refactor it.

  1. export options
  2. remove params type in comment
  3. rename option to options
  4. default optoins instead of optional options

closed time in 6 days

axetroy

create barnchaxetroy/deno_std

branch : ci

created branch time in 6 days

pull request commentdenoland/deno_std

feat: add ensureSymlink/ensureSymlinkSync for fs modules

I don't think Should skip is a good solution. I'd test assertThrows(ensureSymlink()) if deno.platform === 'win'

or should wait for support windows?

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

feat: add remove/removeSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+// It should have its own interface in case there will be new option field in the future+/* eslint-disable-next-line @typescript-eslint/no-empty-interface */+export interface RemoveOptions extends Deno.RemoveOption {}++/* Removes the named file or directory. alias for `Deno.remove` */+export async function remove(+  path: string,+  options: RemoveOptions = {}+): Promise<void> {+  await Deno.remove(path, options);

yes. see here https://github.com/denoland/deno_std/issues/261#issuecomment-472533659

axetroy

comment created time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 785a9825b822bdd1ed08ddd83b5528f74ba18ddd

test: skip test in windwos.

view details

push time in 6 days

issue commentdenoland/deno_std

implement fs-extra api for fs modules

@zekth ok, you do it

axetroy

comment created time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 934150f86286a33f59660d6032a8d338932f7eb7

update docs for methods

view details

push time in 6 days

Pull request review commentdenoland/deno_std

feat: add writeJson/writeJsonSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+/* eslint-disable @typescript-eslint/no-explicit-any */

Should we ignore any warning?

I didn't find the answer in the style guide.

But it seems that the other modules have not ignored it.

axetroy

comment created time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 2ad700fc20694f0591bf7fe5d884546910f729de

fix: rename syslink to symlink

view details

push time in 6 days

Pull request review commentdenoland/deno_std

feat: add ensureSyslink/ensureSyslinkSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import { test } from "../testing/mod.ts";+import {+  assertEquals,+  assertThrows,+  assertThrowsAsync+} from "../testing/asserts.ts";+import { ensureSyslink, ensureSyslinkSync } from "./ensure_syslink.ts";+import * as path from "./path/mod.ts";++const testdataDir = path.resolve("fs", "testdata");++test(async function ensureSyslinkIfItNotExist() {+  const testDir = path.join(testdataDir, "link_file_1");+  const testFile = path.join(testDir, "test.txt");++  assertThrowsAsync(async () => {+    await ensureSyslink(testFile, path.join(testDir, "test1.txt"));+  });++  assertThrowsAsync(async () => {+    await Deno.stat(testFile).then(() => {+      throw new Error("test file should exists.");+    });+  });+});++test(function ensureSyslinkSyncIfItNotExist() {+  const testDir = path.join(testdataDir, "link_file_2");+  const testFile = path.join(testDir, "test.txt");++  assertThrows(() => {+    ensureSyslinkSync(testFile, path.join(testDir, "test1.txt"));+  });++  assertThrows(() => {+    Deno.statSync(testFile);+    throw new Error("test file should exists.");+  });+});++test(async function ensureSyslinkIfItExist() {

make sense.

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

feat: add copy/copySync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";+import { ensureDir, ensureDirSync } from "./ensure_dir.ts";++export interface CopyOptions {+  /* overwrite existing file or directory, default is false */+  overwrite?: boolean;+}++function isSrcSubdir(src: string, dest: string): boolean {

there are two isSrcSubdir method in the fs modules

Should it be extracted into path modules?

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

feat: add copy/copySync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";+import { ensureDir, ensureDirSync } from "./ensure_dir.ts";++export interface CopyOptions {+  /* overwrite existing file or directory, default is false */+  overwrite?: boolean;+}++function isSrcSubdir(src: string, dest: string): boolean {+  const srcArray = src.split(path.sep);+  const destArray = dest.split(path.sep);++  return srcArray.reduce((acc, current, i) => {+    return acc && destArray[i] === current;+  }, true);+}++/* copy file to dest */+async function copyFile(src: string, dest: string): Promise<void> {+  await Deno.copyFile(src, dest);+}++function copyFileSync(src: string, dest: string): void {+  Deno.copyFileSync(src, dest);+}++/* copy link to dest */+/* eslint-disable-next-line @typescript-eslint/no-unused-vars */+async function copyLink(src: string, dest: string): Promise<void> {+  throw new Error("Not implemented copy link");

Link is not supported in the current

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

Added read file str

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.++/**+ * Read file synchronously and output it as a string.+ *+ * @param filename File to read+ * @param [encoding="utf-8"] encoding Encoding of the file+ */+export function readFileStrSync(+  filename: string,+  encoding: string = "utf-8"+): string {+  const decoder = new TextDecoder(encoding);+  return decoder.decode(Deno.readFileSync(filename));+}++/**+ * Read file and output it as a string.+ *+ * @param filename File to read+ * @param [encoding="utf-8"] Encoding of the file+ */+export async function readFileStr(

@zekth fs module is used to add some quick and easy methods

Returning a string is the most common method for developer

In most cases, deno.readFile returns the Reader is not commonly used

So I don’t think these conflicts.

zekth

comment created time in 6 days

Pull request review commentdenoland/deno_std

Added read file str

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.++/**+ * Read file synchronously and output it as a string.+ *+ * @param filename File to read+ * @param [encoding="utf-8"] encoding Encoding of the file+ */+export function readFileStrSync(+  filename: string,+  encoding: string = "utf-8"+): string {+  const decoder = new TextDecoder(encoding);+  return decoder.decode(Deno.readFileSync(filename));+}++/**+ * Read file and output it as a string.+ *+ * @param filename File to read+ * @param [encoding="utf-8"] Encoding of the file+ */+export async function readFileStr(+  filename: string,+  encoding: string = "utf-8"

encoding should be an options object in case of other fields in the future. eg flag

  options: ReadOptions = {}
zekth

comment created time in 6 days

Pull request review commentdenoland/deno_std

Added read file str

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.++/**+ * Read file synchronously and output it as a string.+ *+ * @param filename File to read+ * @param [encoding="utf-8"] encoding Encoding of the file+ */+export function readFileStrSync(+  filename: string,+  encoding: string = "utf-8"+): string {+  const decoder = new TextDecoder(encoding);+  return decoder.decode(Deno.readFileSync(filename));+}++/**+ * Read file and output it as a string.+ *+ * @param filename File to read+ * @param [encoding="utf-8"] Encoding of the file+ */+export async function readFileStr(

how about

export async function readFile(
zekth

comment created time in 6 days

issue commentaxetroy/anti-redirect

pocket重新定向问题

感谢你的反馈

能不能给个具体的页面,或者未去除跳转前的链接

maxduke

comment created time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 2aa835cd43d8c7c21fddb778a2e92b56f7dfc2f6

fix: eslint error

view details

push time in 6 days

PR opened denoland/deno_std

feat: add copy/copySync for fs modules

part of #261

+522 -0

0 comment

6 changed files

pr created time in 6 days

create barnchaxetroy/deno_std

branch : copy

created branch time in 6 days

Pull request review commentdenoland/deno_std

feat: add move/moveSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";+import { exists, existsSync } from "./exists.ts";++interface MoveOptions {

@zekth see #274

axetroy

comment created time in 6 days

PR opened denoland/deno_std

feat: add remove/removeSync for fs modules

part of #261

+101 -0

0 comment

3 changed files

pr created time in 6 days

create barnchaxetroy/deno_std

branch : remove

created branch time in 6 days

Pull request review commentdenoland/deno_std

feat: add writeJson/writeJsonSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+/* eslint-disable @typescript-eslint/no-explicit-any */

Should we ignore any warning?

I didn't find the answer in the style guide.

But it seems that the other modules have not ignored it.

axetroy

comment created time in 6 days

created tagaxetroy/vscode-comment-autocomplete

tagv0.4.2

An VSCODE extension to autocomplete the comment.

created time in 6 days

push eventaxetroy/vscode-comment-autocomplete

axetroy

commit sha 18cdafb5ac03055f96ef0ef092550379f201c21b

feat: add eslint comment for ts file

view details

axetroy

commit sha c53ff59fadb200d6f6df2514d58cffaf4a7aca54

0.4.2

view details

push time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 61f3da7a9b53e7cb6e2bade1235fc4f7d8a33f43

fix: eslint error

view details

push time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 76166f51d0ffa7f2b97639d50f0df5956c688617

styles: format the fs/mod.ts

view details

push time in 6 days

issue openeddenoland/deno_std

refactor fs to following style guide

here to remind me refactor it.

  1. export options
  2. remove params type for comment
  3. rename option to options
  4. default optoins instead of optional options

created time in 6 days

issue closeddenoland/deno_std

discuss: all test file's name should end with _test.ts

In order to maintain the uniformity of the project style, the test file should end with _test.ts

This also helps the third-party library to detect whether it is a test file.

Other test-tools/plugins/extensions will definitely appear in the future.

So should files like this be renamed?

https://github.com/denoland/deno_std/blob/master/ws/test.ts https://github.com/denoland/deno_std/blob/master/textproto/test.ts https://github.com/denoland/deno_std/blob/master/testing/test.ts https://github.com/denoland/deno_std/blob/master/testing/test.ts https://github.com/denoland/deno_std/blob/master/media_types/test.ts https://github.com/denoland/deno_std/blob/master/log/test.ts ...

Make it a de facto standard

Avoid naming confusion like the nodejs community

closed time in 6 days

axetroy

issue commentdenoland/deno_std

discuss: all test file's name should end with _test.ts

It seems that this is not an urgent matter, so close it.

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

feat: add writeJson/writeJsonSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";++interface WriteJsonOption {+  spaces?: number | string;+  replacer?: (number | string)[] | null | Replacer;+}++type Replacer = (key: string, value: any) => any;++/**+ * Writes an object to a JSON file.+ * @export+ * @param {string} filePath+ * @param {*} object+ * @param {WriteJsonOption} [options]+ * @returns {Promise<void>}

I think this interface is obvious and there is no need to explain the parameters. So I only leave a single line comment

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

feat: add writeJson/writeJsonSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";++interface WriteJsonOption {+  spaces?: number | string;+  replacer?: (number | string)[] | null | Replacer;

Good point. I remove it.

axetroy

comment created time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha 9523765ebe92ae76277f3ecac5982fcd1fa18dcf

refactor: reanme WriteJsonOption to WriteJsonOptions

view details

push time in 6 days

push eventaxetroy/deno_std

Kitson Kelly

commit sha bb51bce49b7da88a70b4599e0a5e422972334d58

Rename WriteJsonOption to WriteJsonOptions Co-Authored-By: axetroy <troy450409405@gmail.com>

view details

push time in 6 days

push eventaxetroy/deno_std

axetroy

commit sha ef73f2e3e157e5bbd09f1002b206d5d476b2911b

styles: follow the style guide

view details

push time in 6 days

Pull request review commentdenoland/deno_std

feat: add writeJson/writeJsonSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";++interface WriteJsonOption {+  spaces?: number | string;+  replacer?: (number | string)[] | null | Replacer;+}++type Replacer = (key: string, value: any) => any;++/**+ * Writes an object to a JSON file.+ * @export+ * @param {string} filePath+ * @param {*} object+ * @param {WriteJsonOption} [options]+ * @returns {Promise<void>}+ */+export async function writeJson(+  filePath: string,+  object: any,+  options?: WriteJsonOption

@bartlomieju yes. my bad

axetroy

comment created time in 6 days

Pull request review commentdenoland/deno_std

feat: add writeJson/writeJsonSync for fs modules

+// Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.+import * as path from "./path/mod.ts";++interface WriteJsonOption {+  spaces?: number | string;+  replacer?: (number | string)[] | null | Replacer;

This is the definition of JSON.stringify

replacer can be null

    /**
      * Converts a JavaScript value to a JavaScript Object Notation (JSON) string.
      * @param value A JavaScript value, usually an object or array, to be converted.
      * @param replacer A function that transforms the results.
      * @param space Adds indentation, white space, and line break characters to the return-value JSON text to make it easier to read.
      */
    stringify(value: any, replacer?: (key: string, value: any) => any, space?: string | number): string;
    /**
      * Converts a JavaScript value to a JavaScript Object Notation (JSON) string.
      * @param value A JavaScript value, usually an object or array, to be converted.
      * @param replacer An array of strings and numbers that acts as a approved list for selecting the object properties that will be stringified.
      * @param space Adds indentation, white space, and line break characters to the return-value JSON text to make it easier to read.
      */
    stringify(value: any, replacer?: (number | string)[] | null, space?: string | number): string;
axetroy

comment created time in 6 days

issue commentdenoland/deno_std

discuss: all test filename should end with _test.ts

@zekth Already confirmed?

@j-f1 I prefer to {entry file}_test.ts. It seems that the entry file is not be mod.ts

axetroy

comment created time in 6 days

issue openeddenoland/deno_std

discuss: all test filename should end with _test.ts

In order to maintain the uniformity of the project style, the test file should end with _test.ts

This also helps the test tool to detect whether it is a test file. (other test tools will definitely appear in the future)

So should files like this be renamed?

https://github.com/denoland/deno_std/blob/master/ws/test.ts https://github.com/denoland/deno_std/blob/master/textproto/test.ts https://github.com/denoland/deno_std/blob/master/testing/test.ts https://github.com/denoland/deno_std/blob/master/testing/test.ts https://github.com/denoland/deno_std/blob/master/media_types/test.ts https://github.com/denoland/deno_std/blob/master/log/test.ts ...

created time in 6 days

issue commentdenoland/deno_std

implement fs-extra api for fs modules

I think we should add this API alias to Deno.remove avoid to switch between Deno and fs API

fs.remove(); // same with Deno.remove()
axetroy

comment created time in 6 days

more