Skip to content

Fix command syntax for installing gcc on Mac OS#705

Merged
darrylong merged 1 commit into
PreferredAI:masterfrom
darrylong:docs/readme-gcc-update
Jun 23, 2026
Merged

Fix command syntax for installing gcc on Mac OS#705
darrylong merged 1 commit into
PreferredAI:masterfrom
darrylong:docs/readme-gcc-update

Conversation

@darrylong

Copy link
Copy Markdown
Member

Description

This pull request makes a minor correction to the installation instructions in the README.md file, replacing the use of the pipe (|) with a logical AND (&&) to ensure both brew install gcc and brew link gcc run successfully.

Related Issues

Checklist:

  • I have added tests.
  • I have updated the documentation accordingly.
  • I have updated README.md (if you are adding a new model).
  • I have updated examples/README.md (if you are adding a new example).
  • I have updated datasets/README.md (if you are adding a new dataset).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 corrects the macOS installation command in README.md so the Homebrew gcc install and link steps are executed sequentially (instead of piping output), aligning the documentation with the intended setup for OpenMP-capable compilation.

Changes:

  • Replace brew install gcc | brew link gcc with brew install gcc && brew link gcc in the macOS OpenMP compiler guidance.

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

@qtuantruong qtuantruong left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thank you

@darrylong darrylong added the docs Documentation (Readme, readthedocs) related label Jun 23, 2026
@darrylong darrylong merged commit 76d18dc into PreferredAI:master Jun 23, 2026
19 checks passed
@darrylong darrylong deleted the docs/readme-gcc-update branch June 23, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation (Readme, readthedocs) related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants