-
Notifications
You must be signed in to change notification settings - Fork 82
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
Remove process
dependency
#757
Changes from all commits
b62c2d4
448cdd9
c0d1a90
798600e
dd0d037
81412e0
5b4d23f
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,25 +1,30 @@ | ||
{ | ||
"private": true, | ||
"description": "HASH monorepo", | ||
"workspaces": [ | ||
"packages/blocks/*", | ||
"packages/engine/apache-arrow-js-bundle", | ||
"packages/hash/api", | ||
"packages/hash/backend-utils", | ||
"packages/hash/datastore", | ||
"packages/hash/design-system", | ||
"packages/hash/eslint-config", | ||
"packages/hash/frontend", | ||
"packages/hash/integration", | ||
"packages/hash/playwright", | ||
"packages/hash/realtime", | ||
"packages/hash/search-loader", | ||
"packages/hash/shared", | ||
"packages/hash/task-executor", | ||
"packages/hash/tsconfig", | ||
"sites/hashai", | ||
"sites/hashdev" | ||
], | ||
"workspaces": { | ||
"packages": [ | ||
"packages/blocks/*", | ||
"packages/engine/apache-arrow-js-bundle", | ||
"packages/hash/api", | ||
"packages/hash/backend-utils", | ||
"packages/hash/datastore", | ||
"packages/hash/design-system", | ||
"packages/hash/eslint-config", | ||
"packages/hash/frontend", | ||
"packages/hash/integration", | ||
"packages/hash/playwright", | ||
"packages/hash/realtime", | ||
"packages/hash/search-loader", | ||
"packages/hash/shared", | ||
"packages/hash/task-executor", | ||
"packages/hash/tsconfig", | ||
"sites/hashai", | ||
"sites/hashdev" | ||
], | ||
"nohoist": [ | ||
"**/ts-invariant" | ||
] | ||
}, | ||
"scripts": { | ||
"codegen": "yarn workspace @hashintel/hash-api codegen && yarn workspace @hashintel/hash-frontend codegen && yarn workspace @hashintel/hash-integration codegen && yarn workspace @hashintel/hash-shared codegen", | ||
"dev": "concurrently \"yarn:dev:backend\" \"yarn:dev:frontend\"", | ||
|
@@ -65,7 +70,7 @@ | |
"node-fetch": "^2.6.7" | ||
}, | ||
"devDependencies": { | ||
"check-dependency-version-consistency": "2.0.0", | ||
"check-dependency-version-consistency": "3.0.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. Needed to support package.json → workspaces → nohoist |
||
"concurrently": "7.0.0", | ||
"cross-env": "7.0.3", | ||
"dotenv-flow": "3.2.0", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -33,7 +33,8 @@ | |
"@mui/material": "5.4.3", | ||
"@rjsf/core": "4.1.1", | ||
"@rjsf/material-ui": "4.1.1", | ||
"@sentry/nextjs": "6.17.9", | ||
"@sentry/nextjs": "7.5.1", | ||
"@sentry/react": "7.5.1", | ||
"@svgr/webpack": "6.2.1", | ||
"@twind/next": "1.0.9", | ||
"ajv": "8.11.0", | ||
|
@@ -49,10 +50,9 @@ | |
"jsonschema": "1.4.0", | ||
"lodash": "4.17.21", | ||
"material-ui-popup-state": "2.0.0", | ||
"next": "12.1.0", | ||
"next": "12.2.0", | ||
"next-transpile-modules": "9.0.0", | ||
"pluralize": "8.0.0", | ||
"process": "0.11.10", | ||
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. 🧹 |
||
"prosemirror-collab": "1.2.2", | ||
"prosemirror-commands": "1.2.1", | ||
"prosemirror-dev-tools": "3.1.0", | ||
|
@@ -82,7 +82,7 @@ | |
"@graphql-codegen/fragment-matcher": "2.0.1", | ||
"@graphql-codegen/typescript": "1.22.1", | ||
"@graphql-codegen/typescript-operations": "1.18.0", | ||
"@next/bundle-analyzer": "12.1.0", | ||
"@next/bundle-analyzer": "12.2.0", | ||
"@testing-library/dom": "8.14.0", | ||
"@testing-library/jest-dom": "5.16.4", | ||
"@testing-library/react": "12.1.5", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
import * as Sentry from "@sentry/nextjs"; | ||
import * as Sentry from "@sentry/react"; | ||
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 is a non-conventional way of bundling Next.js. Looks like a separate entry point affects webpack features in a custom bundle and it’s impossible to use 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 there Nextjs-specific features we miss out on when switching to 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. Hard to say, but because |
||
import { | ||
BlockProtocolAggregateEntitiesFunction, | ||
BlockProtocolAggregateEntityTypesFunction, | ||
|
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.
The new configuration of dependencies and sub-dependencies provided
@apollo/client
with a wrong version ofts-invariant
, which caused the crash. Removingnohoist
will be possible after upgrading@apollo/client
andapollo-server-express
and thus avoid mutually incompatiblets-invariant
sub-dependencies. This will be done in #757.Some detail on the issue: apollographql/apollo-client#9094 (comment)