October 10-16, 2021 in Darker: Optimizing Git integration and exploring performance enhancements #662
akaihola
announced in
Announcements
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
This week, we focused on improving Git integration and exploring performance optimizations. We merged a pull request (#167) that extended unit tests for the
git_get_content_at_revision()
function, enhancing our test coverage for Git-related functionality. This included tests for file reading and ensuring Windows compatibility in Git tests.Discussions continued around the SequenceMatcher's suboptimal chunkification (#206), with experiments on adjusting junk heuristics and exploring alternative matching algorithms. We considered delegating diff computation to the
git
binary, which could potentially offer better performance and tuning for code. Additionally, we made progress on performance improvements with the introduction of an ASTVerifier and caching of intermediate data during bisect (#214), addressing issues raised in #211.On the community front, we addressed concerns about using isort with pre-commit hooks (#224). A solution was found by adding
additional_dependencies
to the pre-commit configuration, ensuring that necessary dependencies like Black and isort are installed in the pre-commit environment. We also discussed improving Darker's behavior as a pre-commit hook, particularly regarding its handling of linter output and exit codes (#223), considering options to make it more consistent with pre-commit expectations.Beta Was this translation helpful? Give feedback.
All reactions