-
Notifications
You must be signed in to change notification settings - Fork 1
setup repo with yarn v4 #4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 9 commits
364d42c
1286253
9955d0c
2a8ce35
c4219ec
218116d
4c39203
6959653
b12544b
953ef95
600492e
30c8f7e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| { | ||
| "node": "14", | ||
| "node": "20", | ||
| "packages": ["packages/core", "packages/react"], | ||
| "sandboxes": ["7njqn", "2uwsq"] | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,22 @@ | ||
| name: CI | ||
| on: push | ||
| on: | ||
| push: | ||
| branches: | ||
| - canary | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are we still using this branch?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we will not be using it actively, but since this is the branch our fork is based off of, we should keep it just in case we need to pull in changes from upstream |
||
| - main | ||
| pull_request: | ||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: actions/setup-node@v2 | ||
| - uses: actions/checkout@v4 | ||
| - name: Enable Corepack | ||
| run: corepack enable && yarn --version | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: '16' | ||
| node-version: '20' | ||
| cache: 'yarn' | ||
| - name: Install modules | ||
| run: yarn | ||
| run: yarn install --immutable | ||
| - name: Run tests | ||
| run: yarn test | ||
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| v20.17.0 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also just curious, why do we need a node version for this package?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this just ensures consistency across builds by ensuring all builds were generated using the same tooling |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| nodeLinker: node-modules | ||
|
|
||
| compressionLevel: 0 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,13 +3,14 @@ | |
| "name": "stitches", | ||
| "type": "module", | ||
| "license": "MIT", | ||
| "packageManager": "[email protected]", | ||
| "contributors": [ | ||
| "Pedro Duarte", | ||
| "Abdulhadi Alhallak <[email protected]>", | ||
| "Jonathan Neal" | ||
| ], | ||
| "scripts": { | ||
| "build:test:types": "yarn workspace @stitches/test build", | ||
| "build:test:types": "yarn workspace @sigmacomputing/stitches-test build", | ||
| "build": "node .task/build.js", | ||
| "build:watch": "node .task/build.js --watch", | ||
| "lint": "node .task/lint.js", | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| traces/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to change this? I figure we're not actually using this for CI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah we can delete this since we aren't using codesandbox