Skip to content

Conversation

@furtib
Copy link
Contributor

@furtib furtib commented Oct 21, 2025

Why:
Currently, we use a different, inferior implementation for generating compile_commands.json in the pre_file rule.
We should use the same as in the monolithic rule. This would prevent us from having to maintain both rules simultaneously.

What:
Replaces the compile_commands.json generating implementation in the pre_file rule.
Fixes the unittest for compile_commands, the per_file compile_commands.json location changed

Addresses:
None

@furtib furtib requested review from Szelethus and nettle October 21, 2025 11:41
@furtib furtib self-assigned this Oct 21, 2025
@furtib furtib added enhancement New feature or request non-functional change ☮️ The patch doesn't change any functionality, e.g. refactoring, documentation, test-only. labels Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request non-functional change ☮️ The patch doesn't change any functionality, e.g. refactoring, documentation, test-only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant