Skip to content

Remove stale CLI references from build.sh#496

Merged
SohamDas2021 merged 1 commit into
mainfrom
fix/build-sh-stale-cli-ref
Jun 5, 2026
Merged

Remove stale CLI references from build.sh#496
SohamDas2021 merged 1 commit into
mainfrom
fix/build-sh-stale-cli-ref

Conversation

@SohamDas2021
Copy link
Copy Markdown
Contributor

@SohamDas2021 SohamDas2021 commented Jun 4, 2026

📖 Description

Remove stale CLI references from build.sh
Fixes #486

🔗 References

🔍 Validation

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task

Microsoft reviewers: PR builds don't auto-run (ADO policy). Comment /azp run
to start MXC-PR-Build. See docs/pull-requests.md.

Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings June 4, 2026 20:56
@SohamDas2021 SohamDas2021 requested a review from a team as a code owner June 4, 2026 20:56
@SohamDas2021 SohamDas2021 requested review from bbonaby and huzaifa-d June 4, 2026 20:57
Copy link
Copy Markdown

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 updates the Linux build.sh script to stop referencing/building a TypeScript CLI that no longer exists in the repository, aligning the script with the current repo layout and fixing the ./build.sh failure reported in #486.

Changes:

  • Remove the unused CLI_DIR variable and the CLI build steps from build.sh.
  • Update script help text and header comments to refer to building only the TypeScript SDK (not “SDK/CLI”).

@SohamDas2021
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@SohamDas2021 SohamDas2021 removed the request for review from huzaifa-d June 5, 2026 02:15
@SohamDas2021 SohamDas2021 merged commit c1027bc into main Jun 5, 2026
22 checks passed
@SohamDas2021 SohamDas2021 deleted the fix/build-sh-stale-cli-ref branch June 5, 2026 03:11
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.

CLI directory does not exist

4 participants