Skip to content

chore(js/evaluators): remove unnecessary path dependency #3587

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore(js/evaluators): remove unnecessary path dependency #3587

wants to merge 1 commit into from
+0 −4

Conversation

@10kc-emilyk
Copy link

@10kc-emilyk 10kc-emilyk commented Sep 17, 2025

path is a core node module, and adding it as a dependency is pointless, as node (and other compatible runtimes) will always use the core module

Description here... Help the reviewer by:

  • linking to an issue that includes more details
  • if it's a new feature include samples of how to use the new feature
  • (optional if issue link is provided) if you fixed a bug include basic bug details

Checklist (if applicable):
@10kc-emilyk
chore(js/evaluators): remove unnecessary path dependency
3b11b80 
`path` is a core node module, and adding it as a dependency is pointless, as node (and other compatible runtimes) will always use the core module
@10kc-emilyk 10kc-emilyk force-pushed the remove-unnecessary-path-dependency branch from b24c688 to 3b11b80 Compare September 17, 2025 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config js

1 participant

@10kc-emilyk