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

Fix bugs in translation of PM_RESCUE_NODE #317

Merged
merged 5 commits into from
Nov 5, 2024
Merged

Fix bugs in translation of PM_RESCUE_NODE #317

merged 5 commits into from
Nov 5, 2024

Commits on Nov 5, 2024

  1. Fix edge case where PM_RESCUE_NODE has empty body

    @egiurleo
    egiurleo committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    f86eda5 View commit details
    Browse the repository at this point in the history

  2. Only do dynamic constant assignment workaround in specific cases 

    Only do the dynamic constant assignment workaround when translating
a constant from a `PM_CONSTANT_WRITE_NODE` and
`PM_CONSTANT_PATH_NODE`.
    @egiurleo
    egiurleo committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    92c51ee View commit details
    Browse the repository at this point in the history

  3. Test dynamic constant workaround for all constant assignment types

    @egiurleo
    egiurleo committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    88d6459 View commit details
    Browse the repository at this point in the history

  4. Add more test cases to rescue prism regression tests

    @egiurleo
    egiurleo committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    8f8bc8f View commit details
    Browse the repository at this point in the history

  5. Handle all constant path assignment node types

    @egiurleo
    egiurleo committed Nov 5, 2024
    Configuration menu
    Copy the full SHA
    fcc8a72 View commit details
    Browse the repository at this point in the history