denoland / deno-docs

Docusaurus site for a unified Deno docs experience
https://docs.deno.com
MIT License
45 stars 79 forks source link

no https specifiers that are not using deno.land/x #443

Open ry opened 2 months ago

ry commented 2 months ago

For example, here is a code snippet that uses x.nest.land https://docs.deno.com/runtime/manual/basics/modules/#remote-import

import {
  add,
  multiply,
} from "https://x.nest.land/ramda@0.27.0/source/index.js";

function totalCost(outbound: number, inbound: number, tax: number): number {
  return multiply(add(outbound, inbound), tax);
}

console.log(totalCost(19, 31, 1.2));
console.log(totalCost(45, 27, 1.15));

switch all https imports to https://deno.land/x imports so we can ensure they continue to work

(also that code sample has an annoying amount of comments in it - those should be remove too)