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

Fix bugs in lock diff #616

Merged

psakievich merged 2 commits into main from b/lock-diff-bugs

Dec 20, 2024

+2 −1

Collaborator

psakievich commented Sep 10, 2024

The lock-diff command came out of a project that was using it in a specific way. Several bugs have been found from more general use.

Outstanding issues:

Multiple specs: this tool was mainly written for unified env's but compilers as deps is leading to a lot more repeated specs in the graph. Need better logic/generalization for diffing the graph
Testing: add some basic unit-tests so this can be expanded

psakievich added 2 commits

September 10, 2024 06:50


          Fix bugs in lock diff

0d50d6d


          Merge branch 'main' into b/lock-diff-bugs

315f54f

psakievich merged commit 45134f6 into main

25 of 26 checks passed

psakievich deleted the b/lock-diff-bugs branch

December 20, 2024 22:51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet