Skip to content
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

chore: use NPM workspaces #9562

Merged
merged 4 commits into from
Jun 3, 2023
Merged

chore: use NPM workspaces #9562

merged 4 commits into from
Jun 3, 2023

Conversation

frbuceta
Copy link
Contributor

No description provided.

@frbuceta frbuceta self-assigned this May 23, 2023
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch 4 times, most recently from b34befd to 80d2bc7 Compare May 24, 2023 00:09
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch 2 times, most recently from 4e59738 to f9c676b Compare May 31, 2023 09:15
@frbuceta
Copy link
Contributor Author

frbuceta commented May 31, 2023

@shubhamp-sf Do you know what might be going on here?

Error: Please install sqlite3 package manually

DriverPackageNotInstalledError: SQLite package has not been found installed. Try to install it: npm install sqlite3 --save

@shubhamp-sf
Copy link
Contributor

shubhamp-sf commented May 31, 2023

@frbuceta I see the code changes removed package-lock json and lerna bootstrap step. I tried it locally but it didn't install packages for me. So sequelize and typeorm are looking for that package and not finding it.

@shubhamp-sf
Copy link
Contributor

Btw why are we moving onto NPM workspaces here? Is there any discussion thread I can go through?

@frbuceta
Copy link
Contributor Author

Btw why are we moving onto NPM workspaces here? Is there any discussion thread I can go through?

There is no discussion but I am starting to prepare for migration because of this note on Lerna.

image

@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch from f0e9fef to f9c676b Compare May 31, 2023 10:00
@frbuceta
Copy link
Contributor Author

Also, the project installs the dependencies super fast and that makes the tests in github run much faster.

@shubhamp-sf
Copy link
Contributor

Oh okay, then IMO the only culprit can be require() calls in both typeorm and sequelize for the reason that they require it conditionally It is unable to check them in correct node_modules.

@frbuceta
Copy link
Contributor Author

Oh okay, then IMO the only culprit can be require() calls in both typeorm and sequelize for the reason that they require it conditionally It is unable to check them in correct node_modules.

Yes, that's right. The strange thing is that it only fails in GitHub Actions.

Signed-off-by: Francisco Buceta <frbuceta@gmail.com>
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch from 50dea93 to 8a4aea6 Compare June 1, 2023 20:11
Signed-off-by: Francisco Buceta <frbuceta@gmail.com>
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch 9 times, most recently from 7856d55 to 3c7f36e Compare June 1, 2023 22:39
Signed-off-by: Francisco Buceta <frbuceta@gmail.com>
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch 4 times, most recently from e0e5326 to 7989004 Compare June 2, 2023 07:49
@loopbackio loopbackio deleted a comment from coveralls Jun 2, 2023
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch from 7989004 to dc03796 Compare June 2, 2023 10:01
@frbuceta
Copy link
Contributor Author

frbuceta commented Jun 2, 2023

@raymondfeng Can you help me understand what's going on with webpack, with the web compilation?

image

See #9602

Signed-off-by: Francisco Buceta <frbuceta@gmail.com>
@frbuceta frbuceta force-pushed the frbuceta/use-workspaces branch from 952f360 to d05b64a Compare June 2, 2023 11:50
@frbuceta frbuceta marked this pull request as ready for review June 2, 2023 23:39
@frbuceta frbuceta merged commit f63f42a into master Jun 3, 2023
@frbuceta frbuceta deleted the frbuceta/use-workspaces branch June 3, 2023 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants