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

isolate benchmarked module dependencies #1219

Open
hoeck opened this issue Mar 10, 2024 · 1 comment
Open

isolate benchmarked module dependencies #1219

hoeck opened this issue Mar 10, 2024 · 1 comment

Comments

@hoeck
Copy link
Collaborator

hoeck commented Mar 10, 2024

Having all tested modules share the same node_modules namespace seems to introduce new issues / limits:

  1. dependabot upgrades of Typescript seems to be unreliable: feat: add aeria #1218 (comment)
  2. all modules must share the same Typescript version

The first issue may just be a nuisance, requiring more maintenance work.

The second one though may be more severe, in the worst case requiring us to drop old modules or to wait until all modules are able to use new TS versions.

In #864 we already moved to running each benchmark in its own process. Might be just the next logical step to also isolate node_modules? That also helps isolating the different build / code generation steps required for some modules.

@moltar any feedback?

@moltar
Copy link
Owner

moltar commented Mar 11, 2024

Yes, 100% the issue is there and needs to be fixed.

I wanted to re-design the entire project setup entirely, including this problem if isolation and the umbrella issue is #899, albeit it does not have the meat on that bone there, regarding the details.

I will link to this issue from there for documentation.

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

No branches or pull requests

2 participants