Skip to content

Fix typos in README.md - #263

Open
furtib wants to merge 2 commits into
Ericsson:mainfrom
furtib:doc-fix-no-at-prefix-in-readme
Open

Fix typos in README.md#263
furtib wants to merge 2 commits into
Ericsson:mainfrom
furtib:doc-fix-no-at-prefix-in-readme

Conversation

@furtib

@furtib furtib commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Why:
We want the examples in the README.md to work.

What:

  • Added missing @rules_codechecker prefix in the workspace file example.
  • Fixed syntax of bazel test command.
  • Fixed clang binary example for toolchain target.

Addresses:
none

@furtib
furtib requested a review from Szelethus July 20, 2026 08:26
@furtib furtib self-assigned this Jul 20, 2026
@furtib furtib added the documentation Improvements or additions to documentation label Jul 20, 2026
Comment thread README.md
Comment on lines 509 to 511
clang_tidy = "//example_target:clang-tidy",
clangsa = "//example_target:clang",
codechecker = "//example_target:CodeChecker",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Where is this example_target coming from?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Honestly, in general, this entire section is doing a pretty poor job IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants