Skip to content

Fix CLI parse status filter#4784

Open
barnabasdomozi wants to merge 1 commit intoEricsson:masterfrom
barnabasdomozi:parse_status_rel_path
Open

Fix CLI parse status filter#4784
barnabasdomozi wants to merge 1 commit intoEricsson:masterfrom
barnabasdomozi:parse_status_rel_path

Conversation

@barnabasdomozi
Copy link
Collaborator

The parse status file filter now correctly handles relative file paths in the compile_commands.json file. Additonally, a bug was fixed when the filter was used for nonexistent files.

@barnabasdomozi barnabasdomozi added the CLI 💻 Related to the command-line interface, such as the cmd, store, etc. commands label Feb 19, 2026
@barnabasdomozi barnabasdomozi force-pushed the parse_status_rel_path branch 3 times, most recently from d7024a3 to 83d2ec1 Compare February 19, 2026 15:13
Copy link
Member

@dkrupp dkrupp left a comment

Choose a reason for hiding this comment

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

Looks good, but
can you pls add a testcase with relative path in the compile_commands.json?

The parse status file filter now correctly handles
relative file paths in the compile_commands.json file.
Additonally, a bug was fixed when the filter was used
for nonexistent files.
@barnabasdomozi
Copy link
Collaborator Author

@dkrupp Added test cases in commit ba0fac8.

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

Labels

CLI 💻 Related to the command-line interface, such as the cmd, store, etc. commands

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments