Skip to content

Remove package-lock.json.#128

Merged
manugoyal merged 1 commit intomainfrom
manu/remove-package-lock
Mar 22, 2025
Merged

Remove package-lock.json.#128
manugoyal merged 1 commit intomainfrom
manu/remove-package-lock

Conversation

@manugoyal
Copy link
Contributor

We use pnpm, which maintains a pnpm-lock.yaml. So this file is unnecessary. It was being flagged by our dependabot for old packages.

We use `pnpm`, which maintains a `pnpm-lock.yaml`. So this file is
unnecessary. It was being flagged by our dependabot for old packages.
@manugoyal manugoyal requested a review from ibolmo March 22, 2025 17:51
@github-actions
Copy link

github-actions bot commented Mar 22, 2025

Braintrust eval report

Autoevals (manu/remove-package-lock-1742665914)

Score Average Improvements Regressions
NumericDiff 74% (+28.999999999999996pp) 76 🟢 9 🔴
Start 1742665914.66s - -
End 1742665916.26s - -
Duration 1.66s (-3.9s) 100 🟢 22 🔴
Llm_duration 1.84s (-0.53s) 1 🟢 21 🔴
Prompt_tokens 279.25tok (-478.82tok) 74 🟢 45 🔴
Completion_tokens 20.56tok (+3tok) 72 🟢 47 🔴
Total_tokens 299.81tok (-475.82tok) 74 🟢 45 🔴
Estimated_cost 0$ - -

@manugoyal manugoyal merged commit 689f3d6 into main Mar 22, 2025
8 checks passed
@github-actions
Copy link

github-actions bot commented Mar 22, 2025

Braintrust eval report

Autoevals (main-1742666250)

Score Average Improvements Regressions
NumericDiff 74% (+0pp) - -
Start 1742666250.54s - -
End 1742666252.09s - -
Duration 1.54s (-0.12s) 19 🟢 81 🔴
Prompt_tokens 279.25tok (+0tok) - -
Completion_tokens 20.56tok (+0tok) - -
Total_tokens 299.81tok (+0tok) - -

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.

1 participant