Skip to content

Commit

Permalink
Merge pull request #1043 from getguesstimate/pnpm
Browse files Browse the repository at this point in the history
yarn -> pnpm, CI action, CODEOWNERS
  • Loading branch information
berekuk authored Mar 5, 2024
2 parents 9acb6f8 + 47c9e38 commit 5169553
Show file tree
Hide file tree
Showing 5 changed files with 7,087 additions and 6,931 deletions.
26 changes: 18 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,18 +1,28 @@
name: Tests

on: [push, pull_request]
on:
push:
branches:
- main
pull_request:
branches:
- main

jobs:
test:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3
- name: Set up node ${{ matrix.node-version }}
uses: actions/setup-node@v3
- uses: actions/checkout@v4
- uses: pnpm/action-setup@v3
with:
node-version: ${{ matrix.node-version }}
version: 8
- name: Setup Node.js environment
uses: actions/setup-node@v4
with:
node-version: 18
cache: "pnpm"
- name: Install dependencies
run: yarn
- name: Test once
run: yarn ci:test
run: pnpm install
- name: Turbo run
run: pnpm ci:test
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ All models are stored and saved as 'spaces'. The spreadsheet content is all stor

However, while there are a few third party systems, everything will still work with just guesstimate-app, as long as you don't try to log in or save. In practice this means that you can do quite a bit of development, as you can edit any model on the site (just can't save them). It also makes much development quite simple when it's just on the website (not the server).

You can point your local instance to production APIs by setting `NEXT_PUBLIC_API_ENV`: `NEXT_PUBLIC_API_ENV=production yarn dev`.
You can point your local instance to production APIs by setting `NEXT_PUBLIC_API_ENV`: `NEXT_PUBLIC_API_ENV=production pnpm dev`.

## How to run

Expand All @@ -77,8 +77,8 @@ First, make sure that git and node are installed.

`cd guesstimate-app`

`yarn install`
`pnpm install`

`yarn dev`
`pnpm dev`

There are often errors with specific things, but it depends on what is already installed on the computer. Later we could put it in a docker container or something.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"scripts": {
"dev": "next dev",
"start": "next start",
"ci:test": "yarn test",
"ci:test": "pnpm test",
"lint": "eslint . --quiet",
"test": "jest"
},
Expand Down
Loading

0 comments on commit 5169553

Please sign in to comment.