nodejs / nodejs/node

[`util.styleText`] Allow `text` param to be any scalar value (ex. `number`, `bool` etc additionaly to `string`) which will be casted internally to `string`

オープン 初心者向け
#63,841 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

feature request
主要言語
JavaScript
スター
122k
フォーク
37.3k
平均マージ
4日 2時間
マージ済み PR(30日)
283

説明

What is the problem this feature will solve?

Sometimes need to pass as text param for styleText some value that is not string like number, in this case need to explicitly cast value to string which increases boilerplate code to work with styleText easily

What is the feature you are proposing to solve the problem?

Suggestion is to cast text parameter value in styleText method to String(text) if it is not string

What alternatives have you considered?

No response

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず util.styleText の実装と既存のテストを見つけ、次に text パラメータが現在どのように検証または使用されているかを確認します。スカラー値を受け入れて文字列に変換するように動作を更新し、関連するテストを実行して、文字列入力と非文字列入力が期待されるスタイル付き出力を生成することを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
cli
issue の種類
機能追加
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
68/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。