Skip to content

Fix yarn command format in CLAUDE.md to use local yarn version#18553

Merged
harupy merged 2 commits intomasterfrom
copilot/fix-yarn-command-in-claude-md
Oct 28, 2025
Merged

Fix yarn command format in CLAUDE.md to use local yarn version#18553
harupy merged 2 commits intomasterfrom
copilot/fix-yarn-command-in-claude-md

Conversation

Copy link
Contributor

Copilot AI commented Oct 28, 2025

yarn --cwd doesn't respect .yarnrc.yml in the target directory, causing commands to use the global yarn (1.22.22) instead of the project's local version (4.9.1) specified in mlflow/server/js/yarn/releases/yarn-4.9.1.cjs.

Changes

Replaced all yarn --cwd mlflow/server/js commands with (cd mlflow/server/js && yarn ...) to ensure .yarnrc.yml is properly read:

# Before
yarn --cwd mlflow/server/js test

# After
(cd mlflow/server/js && yarn test)

Updated 7 command examples across testing, linting, and type-checking sections.

Original prompt

Replace yarn --cwd mlflow/server/js ... with (cd mlflow/server/js && yarn ...) in CLAUDE.md. yarn --cwd does not use mlflow/server/js/yarn/releases/yarn-4.9.1.cjs. It uses the globally installed yarn, which may lead to version mismatch issues.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@harupy harupy marked this pull request as ready for review October 28, 2025 09:44
@harupy harupy added the rn/none List under Small Changes in Changelogs. label Oct 28, 2025
@github-actions
Copy link
Contributor

@Copilot Thank you for the contribution! Could you fix the following issue(s)?

⚠ Invalid PR template

This PR does not appear to have been filed using the MLflow PR template. Please copy the PR template from here and fill it out.

Co-authored-by: harupy <17039389+harupy@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace yarn command with directory change for version consistency Fix yarn command format in CLAUDE.md to use local yarn version Oct 28, 2025
Copilot AI requested a review from harupy October 28, 2025 09:53
@harupy harupy merged commit 1571ec0 into master Oct 28, 2025
37 of 71 checks passed
@harupy harupy deleted the copilot/fix-yarn-command-in-claude-md branch October 28, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rn/none List under Small Changes in Changelogs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants