Skip to content
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

[Proposal] Refactor resolve fn to be more flexible and extensible #2775

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

[Proposal] Refactor resolve fn to be more flexible and extensible #2775

wants to merge 2 commits into from

Commits on Sep 12, 2022

  1. WIP Refactor resolve fn to be more flexible and extensible 

    Move node-specific resolution logic from `resolve` function to each node type class to allow extending / changing the behaviour in custom types, as well as cleaning up the code a bit
    @mattvague
    mattvague committed Sep 12, 2022
    Configuration menu
    Copy the full SHA
    d8a11dd View commit details
    Browse the repository at this point in the history

  2. Fix lint

    @mattvague
    mattvague committed Sep 12, 2022
    Configuration menu
    Copy the full SHA
    ac31ab2 View commit details
    Browse the repository at this point in the history