Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

apply_fixes can parse report files from a DWYU execution log #255

Merged
merged 1 commit into from
May 26, 2024

Conversation

martis42
Copy link
Owner

For large workspaces discovering the DWYU report files by crawling the bazel-out directory can be quite slow due to an enormous amount of files and directories being present.
To work around this, we enable the apply_fixes script to parse a log file containing the command line output of executing the DWYU aspect. This execution log is then parsed and the DWYU report paths deduced.

@martis42 martis42 force-pushed the apply_fixes_can_get_report_files_from_execution_log branch from 56815ea to a85523a Compare May 26, 2024 10:12
For large workspaces discovering the DWYU report files by crawling the bazel-out
directory can be quite slow due to an enormous amount of files and directories
being present.
To work around this, we enable the apply_fixes script to parse a log file
containing the command line output of executing the DWYU aspect. This execution
log is then parsed and the DWYU report paths deduced.
@martis42 martis42 force-pushed the apply_fixes_can_get_report_files_from_execution_log branch from f065906 to c7d28d2 Compare May 26, 2024 10:54
@martis42 martis42 merged commit 42d0aff into main May 26, 2024
11 checks passed
@martis42 martis42 deleted the apply_fixes_can_get_report_files_from_execution_log branch May 26, 2024 11:21
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