Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

utils.parseArgsの利用法を追記 #1698

Closed
2 of 6 tasks
Tracked by #1719
yusuke-omae opened this issue Nov 5, 2023 · 6 comments · Fixed by #1758
Closed
2 of 6 tasks
Tracked by #1719

utils.parseArgsの利用法を追記 #1698

yusuke-omae opened this issue Nov 5, 2023 · 6 comments · Fixed by #1758

Comments

@yusuke-omae
Copy link

yusuke-omae commented Nov 5, 2023

該当ページ

リクエスト/提案内容

当ページではnpmパッケージを利用するチュートリアルとしてcommanderパッケージを利用しています。
ただしnode.js v20.9.0(LTS)ではimport { parseArgs } from 'node:utils'を用いることでnpmパッケージ非依存で当該処理を解決できます。
https://nodejs.org/dist/latest-v20.x/docs/api/util.html#utilparseargsconfig
この関数に関して下記のいずれかの対応があった方がいいと思います。

  • あえて無視していることを記述
  • コラムとして現状のnode.jsには当該関数があることを明記
  • commanderパッケージから置き換える(npmパッケージ使用のチュートリアルを別ページに書く必要がでる)

以上検討していただけるでしょうか。

TODO

Ref #1698 (comment)

@azu azu added the Status: Proposal 提案段階の状態 label Dec 10, 2023
@azu
Copy link
Collaborator

azu commented Dec 10, 2023

- laco: node:test, util.parseArgsどちらもexperimentalっぽい

まだ入れてなかったのは、以前はexperimentalでの実装だったからですね。
node:testも含めて、Node CLIの章を書き直すのが良いと思います。

一応外部モジュールを扱うユースケースではあるので、npm installが必要になるのがmarkedまで後ろにずれてしまうのが気になるところですかね。

@azu

This comment was marked as outdated.

@azu

This comment was marked as outdated.

@azu
Copy link
Collaborator

azu commented Mar 20, 2024

@lacolaco
Copy link
Collaborator

ありがとうございます!

@azu
Copy link
Collaborator

azu commented Mar 23, 2024

@yossydev さん と @jp-knj さんに担当してもらう

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants