SNSystems / dexter

DExTer - Debug Experience Tester
MIT License
33 stars 6 forks source link

Remove annotate-expected-values #63

Closed OCHyams closed 4 years ago

OCHyams commented 5 years ago

The tool is to be removed for the initial upstream patch.

AEV is great in thoery but has not been updated alongside core changes to Dexter so it will need a fair amount of attention to fix.

Removing it reduces clutter in the upstream RFC patch.

TomWeaver18 commented 5 years ago

LGTM

gregbedwell commented 4 years ago

Makes sense. LGTM, assuming that the current conflict is nothing too major.

OCHyams commented 4 years ago

Rebased