Skip to content

Fixed #515: Added bash autocomplete script.#718

Merged
andreasfertig merged 1 commit intomainfrom
fixIssue515
Jun 18, 2025
Merged

Fixed #515: Added bash autocomplete script.#718
andreasfertig merged 1 commit intomainfrom
fixIssue515

Conversation

@andreasfertig
Copy link
Owner

This change cleans up the option handling. All options are now placed in InsightsOptions.def. This patch added a new option --autocomplete which roughly follows Clang's --autocomplete option. Except that it solely prints all available C++ Insights options and exists.

Please note that Clang tooling options like -p are not part of the output. The new script scripts/bash-autocomplete.sh can be downloaded and put into your bash's completion directory, and you can autocomplete C++ Insights and Clang options passable to C++ Insights.

This change cleans up the option handling. All options are now placed in
`InsightsOptions.def`. This patch added a new option `--autocomplete` which
roughly follows Clang's `--autocomplete` option. Except that it solely prints
all available C++ Insights options and exists.

Please note that Clang tooling options like `-p` are not part of the output.
The new script `scripts/bash-autocomplete.sh` can be downloaded and put into
your bash's completion directory, and you can autocomplete C++ Insights and
Clang options passable to C++ Insights.
@codecov
Copy link

codecov bot commented Jun 18, 2025

Codecov Report

Attention: Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 97.08%. Comparing base (bbaff70) to head (0c0a05d).

Files with missing lines Patch % Lines
Insights.cpp 50.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #718      +/-   ##
==========================================
- Coverage   97.29%   97.08%   -0.22%     
==========================================
  Files          21       22       +1     
  Lines        6690     6720      +30     
==========================================
+ Hits         6509     6524      +15     
- Misses        181      196      +15     
Flag Coverage Δ
insights-macos 95.97% <0.00%> (-0.51%) ⬇️
insights-ubuntu-amd64-libcxx-No 98.18% <100.00%> (+<0.01%) ⬆️
insights-ubuntu-amd64-libcxx-Yes 98.16% <100.00%> (+<0.01%) ⬆️
insights-ubuntu-arm64-libcxx-No 98.18% <100.00%> (+<0.01%) ⬆️
insights-ubuntu-arm64-libcxx-Yes 98.16% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andreasfertig andreasfertig merged commit da1e5ec into main Jun 18, 2025
24 of 26 checks passed
@andreasfertig andreasfertig deleted the fixIssue515 branch June 18, 2025 13:36
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.

1 participant