denoland / dotland

[Archived] deno.land website
https://deno.land
MIT License
957 stars 626 forks source link

Indent based codeblocks cause deployment failure #2031

Closed kt3k closed 2 years ago

kt3k commented 2 years ago

The url https://deno.land/std@0.127.0/fmt returns 502: Bad Gateway (DEPLOYMENT_FAILED) error.

Other std submodules don't seem having this issue, for example std/archive works (https://deno.land/std@0.127.0/archive)

ref https://github.com/denoland/deno_std/issues/1969

crowlKats commented 2 years ago
Error rendering page TypeError: Cannot read properties of undefined (reading 'split')
Please report this to https://github.com/markedjs/marked.
    at Renderer.code (https://deno.land/x/gfm@0.1.18/mod.ts:18:25)
    at j.parse (https://cdn.esm.sh/v66/marked@3.0.7/deno/marked.js:48:2211)
    at Function.parse (https://cdn.esm.sh/v66/marked@3.0.7/deno/marked.js:48:1493)
    at p (https://cdn.esm.sh/v66/marked@3.0.7/deno/marked.js:49:2250)
    at render (https://deno.land/x/gfm@0.1.18/mod.ts:45:16)
    at Object.Markdown (file:///Users/crowlkats/projects/denoland/dotland/components/Markdown.tsx:10:18)
    at m (https://cdn.esm.sh/v59/preact-render-to-string@5.1.19/X-ZGVwczpwcmVhY3RAMTAuNS4xNA/deno/preact-render-to-string.js:4:916)
    at m (https://cdn.esm.sh/v59/preact-render-to-string@5.1.19/X-ZGVwczpwcmVhY3RAMTAuNS4xNA/deno/preact-render-to-string.js:9:66)
    at m (https://cdn.esm.sh/v59/preact-render-to-string@5.1.19/X-ZGVwczpwcmVhY3RAMTAuNS4xNA/deno/preact-render-to-string.js:9:66)
    at m (https://cdn.esm.sh/v59/preact-render-to-string@5.1.19/X-ZGVwczpwcmVhY3RAMTAuNS4xNA/deno/preact-render-to-string.js:4:1044)
crowlKats commented 2 years ago

easy reproducible case:

import {render} from "https://deno.land/x/gfm@0.1.18/mod.ts";

console.log(render(`
If you are looking for the documentation proper, skip to:

    "printf: prints formatted output"

and
`));

replacing the string to be written with "normal" MD code blocks fixes it

crowlKats commented 2 years ago

A fix has been merged, keeping this issue open until gfm gets a new release

kt3k commented 2 years ago

@crowlKats Thanks for the fix!

crowlKats commented 2 years ago

Closing as not the case anymore