Skip to content

Commit

Permalink
Prepare for react-router v7 by removing installGlobals and upgrading …
Browse files Browse the repository at this point in the history
…Node and remix deps
  • Loading branch information
justinsilvestre committed Nov 24, 2024
1 parent cd9410a commit 2071c41
Show file tree
Hide file tree
Showing 8 changed files with 1,119 additions and 460 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
with:
cache: npm
cache-dependency-path: ./package.json
node-version: 18
node-version: 20

- name: 📥 Install deps
run: npm install
Expand All @@ -48,7 +48,7 @@ jobs:
with:
cache: npm
cache-dependency-path: ./package.json
node-version: 18
node-version: 20

- name: 📥 Install deps
run: npm install
Expand All @@ -68,7 +68,7 @@ jobs:
with:
cache: npm
cache-dependency-path: ./package.json
node-version: 18
node-version: 20

- name: 📥 Install deps
run: npm install
Expand Down
2 changes: 1 addition & 1 deletion .tool-versions
Original file line number Diff line number Diff line change
@@ -1 +1 @@
nodejs 18.17.0
nodejs 20.9.0
3 changes: 0 additions & 3 deletions cypress/support/create-user.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,11 @@
// and it will log out the cookie value you can use to interact with the server
// as that new user.

import { installGlobals } from "@remix-run/node";
import { parse } from "cookie";

import { createUser } from "~/models/user.server";
import { createUserSession } from "~/session.server";

installGlobals();

async function createAndLogin(email: string) {
if (!email) {
throw new Error("email required for login");
Expand Down
3 changes: 0 additions & 3 deletions cypress/support/delete-user.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,9 @@
// and that user will get deleted

import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
import { installGlobals } from "@remix-run/node";

import { prisma } from "~/db.server";

installGlobals();

async function deleteUser(email: string) {
if (!email) {
throw new Error("email required for login");
Expand Down
Loading

0 comments on commit 2071c41

Please sign in to comment.