Skip to content

Conversation

@jderochervlk
Copy link
Collaborator

  • Changed the reanalyze npm script in package.json to use rescript-tools reanalyze instead of the standalone reanalyze binary.
  • Removed the reanalyze dependency from package.json as it is no longer needed.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 1, 2025

Deploying rescript-lang-org with  Cloudflare Pages  Cloudflare Pages

Latest commit: 91fa0ce
Status: ✅  Deploy successful!
Preview URL: https://b687ae6d.rescript-lang.pages.dev
Branch Preview URL: https://update-reanalyze-command.rescript-lang.pages.dev

View logs

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates from the standalone reanalyze package to using the integrated rescript-tools binary that comes with the rescript package.

  • Updated the reanalyze npm script to use rescript-tools reanalyze instead of the standalone binary
  • Removed the reanalyze package dependency from both package.json and yarn.lock

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updated the reanalyze script command and removed the reanalyze dependency from devDependencies
yarn.lock Removed the reanalyze package entry and its reference from the workspace dependencies list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fhammerschmidt fhammerschmidt merged commit 09c1fc1 into master Dec 1, 2025
8 checks passed
@fhammerschmidt fhammerschmidt deleted the update-reanalyze-command branch December 1, 2025 18:13
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.

3 participants