Closed ascandone closed 3 months ago
The recent changes involve the complete removal of the VisitValueAwareSource
method from the parseVisitor
class. This method previously handled resource address retrieval for accounts within various contexts but has been eliminated, signifying a major shift in resource management within the parsing logic. As a result, the codebase may be undergoing a restructuring that alters how account resources are processed.
Files | Change Summary |
---|---|
components/ledger/internal/machine/script/compiler/source.go |
Removed the VisitValueAwareSource method from the parseVisitor class, which eliminated the logic for retrieving account resource addresses from different contexts. |
🐇 In the code where rabbits play,
A method hopped away today.
No more resource calls to chase,
A new path now, a different space.
With every change, we leap and bound,
In this code, new joys are found! 🥕
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Apparently this is dead code and deleting it doesn't cause any test to break. I cannot find this function used anywhere using lexical search around the repo Should we delete this? Or is it exposed publicly?