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.

Sign up for GitHub

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

Jump to bottom

Cache and move load_commit_diff out of Report lock #784

Merged
merged 1 commit into from
Oct 16, 2024
Merged

Cache and move load_commit_diff out of Report lock #784

merged 1 commit into from
Oct 16, 2024

Commits on Oct 16, 2024

  1. Cache and move load_commit_diff out of Report lock 

    Moves the `load_commit_diff` portion out of `save_report_results`.

This way, the whole commit diff loading can be moved outside of the Report merge lock.
Additionally, the function was decorated with a cache, as the diff for a commit will never change.
    @Swatinem
    Swatinem committed Oct 16, 2024
    Configuration menu
    Copy the full SHA
    80a8138 View commit details
    Browse the repository at this point in the history