Skip to content

Conversation

@navidaminnn
Copy link
Collaborator

I've added the dependency and made the appropriate changes in Manip.takeDiff to utilize the library. Given I'm on Windows and haven't been able to see how the .diff file is currently formatted, I wasn't entirely sure on how to output the diff.

I've attached a screenshot showing how the diff is shown using the library. If it's different from how the diffs look using the diff command, let me know and I'll change it to match it better.

diff-ss

@fhackett
Copy link
Collaborator

fhackett commented Feb 5, 2025

Note for me, so I don't forget: this is pending the removal of .toList from the proposed code, because difflib works better with O(1) indexing structures rather than linked lists, and the output of os.read.lines is an array already.

@fhackett fhackett merged commit 6e7fd0f into DistCompiler:main Feb 16, 2025
8 checks passed
@navidaminnn navidaminnn deleted the diff-dependency branch December 15, 2025 03:51
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.

2 participants