Package Command Should Preserve Map Key Order #369
Conversation
|
Looks like this change broke anchors. Maybe we need to look for another way to support anchors, since the Decode/Encode was an expedient solution that had unwanted side effects (like losing line numbers). |
|
Didn't see that coming lol. I can take another look later to see what I can come up with. |
|
I think I got it to work. I had to modify the test file as well since order needed to be reflected in the expected result (and style). I added a visitor package under cft and used it to manually resolve the alias nodes. I still had to Marshal and Unmarshal the result so line/column numbers get updated on the anchored nodes. Let me know if anything needs changing. |
|
@ericzbeard Thanks for merging this PR in! Does this project release on a certain schedule? I am just wondering when the next release will be with this fix implemented. |
|
@ericzbeard ^^ Any idea when the next release will be? 😃 |
|
I'll get one out today or tomorrow. No schedule, just whenever there are features and fixes ready to go out. |
Issue #, if available:
Fixes #362
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.