diff options
author | Bartek IwaĆczuk <biwanczuk@gmail.com> | 2024-01-24 16:35:47 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-24 16:35:47 +0100 |
commit | 50eaeabefb83042686e271687979c2a0c40cc7e6 (patch) | |
tree | 668423e1c33469ab448c6335793ce964a63e3026 /runtime/js/99_main.js | |
parent | 62786cfebb5c9fe36d0930582951f442bdfe9441 (diff) |
feat: Change deprecation warning formatting (#22080)
Removes weird "frame like" characters and simplifies the output:
```
warning: Use of deprecated "Deno.isatty()" API. This API will be removed in Deno 2.
Stack trace:
at file:///Users/ib/dev/deno/foo.js:2:8
hint: Use `stdStream.isTerminal()` instead.
warning: Use of deprecated "Deno.isatty()" API. This API will be removed in Deno 2.
Stack trace:
at file:///Users/ib/dev/deno/foo.js:7:8
hint: Use `stdStream.isTerminal()` instead.
```
https://github.com/denoland/deno/assets/13602871/7a6e24bf-44ec-4dbf-ac96-2af1db9f2ab9
Diffstat (limited to 'runtime/js/99_main.js')
-rw-r--r-- | runtime/js/99_main.js | 47 |
1 files changed, 15 insertions, 32 deletions
diff --git a/runtime/js/99_main.js b/runtime/js/99_main.js index fe3dab58b..0cfcafbd2 100644 --- a/runtime/js/99_main.js +++ b/runtime/js/99_main.js @@ -143,49 +143,32 @@ function warnOnDeprecatedApi(apiName, stack, ...suggestions) { ALREADY_WARNED_DEPRECATED.add(apiName + stack); console.error( - "%cWarning", - "color: yellow; font-weight: bold;", - ); - console.error( - `%c\u251c Use of deprecated "${apiName}" API.`, - "color: yellow;", - ); - console.error("%c\u2502", "color: yellow;"); - console.error( - "%c\u251c This API will be removed in Deno 2.0. Make sure to upgrade to a stable API before then.", + `%cwarning: %cUse of deprecated "${apiName}" API. This API will be removed in Deno 2.`, "color: yellow;", + "font-weight: bold;", ); + console.error(); + if (stackLines.length > 0) { + console.error("Stack trace:"); + for (let i = 0; i < stackLines.length; i++) { + console.error(` ${StringPrototypeTrim(stackLines[i])}`); + } + console.error(); + } + for (let i = 0; i < suggestions.length; i++) { const suggestion = suggestions[i]; - console.error("%c\u2502", "color: yellow;"); console.error( - `%c\u251c Suggestion: ${suggestion}`, - "color: yellow;", + `%chint: ${suggestion}`, + "font-weight: bold;", ); } if (isFromRemoteDependency) { - console.error("%c\u2502", "color: yellow;"); console.error( - `%c\u251c Suggestion: It appears this API is used by a remote dependency.`, - "color: yellow;", + `%chint: It appears this API is used by a remote dependency. Try upgrading to the latest version of that dependency.`, + "font-weight: bold;", ); - console.error( - "%c\u2502 Try upgrading to the latest version of that dependency.", - "color: yellow;", - ); - } - if (stackLines.length > 0) { - console.error("%c\u2502", "color: yellow;"); - console.error("%c\u2514 Stack trace:", "color: yellow;"); - for (let i = 0; i < stackLines.length; i++) { - console.error( - `%c ${i == stackLines.length - 1 ? "\u2514" : "\u251c"}\u2500 ${ - StringPrototypeTrim(stackLines[i]) - }`, - "color: yellow;", - ); - } } console.error(); } |