Skip to content

Commit

Permalink
Revert "ci: add typechecking for deno (#4715)"
Browse files Browse the repository at this point in the history
This partially reverts commit c6415bb.
  • Loading branch information
mcansh committed Dec 13, 2022
1 parent c4957c1 commit f96b9d1
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 49 deletions.
5 changes: 0 additions & 5 deletions .changeset/clean-clocks-float.md

This file was deleted.

5 changes: 0 additions & 5 deletions .github/workflows/nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,6 @@ jobs:
node-version-file: ".nvmrc"
cache: "yarn"

- name: 🦕 Setup deno
uses: denoland/setup-deno@v1
with:
deno-version: vx.x.x

- name: 📥 Install deps
run: yarn --frozen-lockfile

Expand Down
25 changes: 0 additions & 25 deletions .github/workflows/reusable-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,6 @@ jobs:
node-version-file: ".nvmrc"
cache: "yarn"

- name: 🦕 Setup deno
uses: denoland/setup-deno@v1
with:
deno-version: vx.x.x

- name: 📥 Install deps
run: yarn --frozen-lockfile

Expand Down Expand Up @@ -66,11 +61,6 @@ jobs:
node-version: ${{ matrix.node }}
cache: "yarn"

- name: 🦕 Setup deno
uses: denoland/setup-deno@v1
with:
deno-version: vx.x.x

- name: 📥 Install deps
run: yarn --frozen-lockfile

Expand Down Expand Up @@ -103,11 +93,6 @@ jobs:
node-version: ${{ matrix.node }}
cache: "yarn"

- name: 🦕 Setup deno
uses: denoland/setup-deno@v1
with:
deno-version: vx.x.x

- name: 📥 Install deps
run: yarn --frozen-lockfile

Expand Down Expand Up @@ -139,11 +124,6 @@ jobs:
node-version: ${{ matrix.node }}
cache: "yarn"

- name: 🦕 Setup deno
uses: denoland/setup-deno@v1
with:
deno-version: vx.x.x

- name: 📥 Install deps
run: yarn --frozen-lockfile

Expand Down Expand Up @@ -175,11 +155,6 @@ jobs:
node-version: ${{ matrix.node }}
cache: "yarn"

- name: 🦕 Setup deno
uses: denoland/setup-deno@v1
with:
deno-version: vx.x.x

- name: 📥 Install deps
run: yarn --frozen-lockfile

Expand Down
6 changes: 3 additions & 3 deletions packages/remix-deno/server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ function defaultCacheControl(url: URL, assetsPublicPath = "/build/") {
}

export function createRequestHandler<
Context extends AppLoadContext | undefined = undefined,
Context extends AppLoadContext | undefined = undefined
>({
build,
mode,
Expand Down Expand Up @@ -53,7 +53,7 @@ export async function serveStaticFiles(
cacheControl?: string | ((url: URL) => string);
publicDir?: string;
assetsPublicPath?: string;
},
}
) {
const url = new URL(request.url);

Expand Down Expand Up @@ -84,7 +84,7 @@ export async function serveStaticFiles(
}

export function createRequestHandlerWithStaticFiles<
Context extends AppLoadContext | undefined = undefined,
Context extends AppLoadContext | undefined = undefined
>({
build,
mode,
Expand Down
11 changes: 0 additions & 11 deletions scripts/build.mjs
Original file line number Diff line number Diff line change
@@ -1,22 +1,11 @@
import { spawn } from "cross-spawn";
import glob from "glob";

const args = process.argv.slice(2);
const publish = process.env.CI || args.includes("--publish");
const tsc = process.env.CI || args.includes("--tsc") || publish;
const denoCheck = process.env.CI || args.includes("--deno");

exec("yarn", ["rollup", "-c"])
.then(() => tsc && exec("yarn", ["tsc", "-b"]))
.then(
() =>
denoCheck &&
exec("deno", [
"check",
"--import-map=.vscode/deno_resolve_npm_imports.json",
...glob.sync("packages/remix-deno/**/*.ts"),
])
)
.then(() => publish && exec("node", ["scripts/copy-build-to-dist.mjs"]))
.then(() => process.exit(0))
.catch((err) => {
Expand Down

0 comments on commit f96b9d1

Please sign in to comment.