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(sem): nimsuggest crashing for ill-formed AST #939

Merged
merged 20 commits into from
Oct 5, 2023
Merged

fix(sem): nimsuggest crashing for ill-formed AST #939

merged 20 commits into from
Oct 5, 2023

Commits on Oct 3, 2023

  1. add reportAndForceRaise

    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    4d1159d View commit details
    Browse the repository at this point in the history

  2. replace globalReport in checkSonsLen,checkMinSonsLen with `repo… 

    …rtAndForceRaise`
    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    148945f View commit details
    Browse the repository at this point in the history

  3. add test case

    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    e6ba246 View commit details
    Browse the repository at this point in the history

  4. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 3, 2023
    Configuration menu
    Copy the full SHA
    7401600 View commit details
    Browse the repository at this point in the history

  5. Update nimsuggest/tests/terror_handling.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 3, 2023
    Configuration menu
    Copy the full SHA
    2387747 View commit details
    Browse the repository at this point in the history

  6. follow suggestions

    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    801ec0d View commit details
    Browse the repository at this point in the history

  7. fix test

    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    d6f99f2 View commit details
    Browse the repository at this point in the history

  8. doc comment

    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    7682b9d View commit details
    Browse the repository at this point in the history

  9. fix miss parameter

    @bung87
    bung87 committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    07681ff View commit details
    Browse the repository at this point in the history

  10. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 3, 2023
    Configuration menu
    Copy the full SHA
    b3b5e05 View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2023

  1. follow suggestions

    @bung87
    bung87 committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    e5f5a82 View commit details
    Browse the repository at this point in the history

  2. forward handleReport

    @bung87
    bung87 committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    4962432 View commit details
    Browse the repository at this point in the history

  3. Update nimsuggest/tests/terror_handling.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 4, 2023
    Configuration menu
    Copy the full SHA
    da3fa18 View commit details
    Browse the repository at this point in the history

  4. fix test

    @bung87
    bung87 committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    5714482 View commit details
    Browse the repository at this point in the history

  5. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 4, 2023
    Configuration menu
    Copy the full SHA
    f773808 View commit details
    Browse the repository at this point in the history

  6. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 4, 2023
    Configuration menu
    Copy the full SHA
    9b1d0de View commit details
    Browse the repository at this point in the history

  7. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 4, 2023
    Configuration menu
    Copy the full SHA
    b758ceb View commit details
    Browse the repository at this point in the history

  8. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 4, 2023
    Configuration menu
    Copy the full SHA
    0add81e View commit details
    Browse the repository at this point in the history

  9. follow suggestion

    @bung87
    bung87 committed Oct 4, 2023
    Configuration menu
    Copy the full SHA
    bc3b9cb View commit details
    Browse the repository at this point in the history

Commits on Oct 5, 2023

  1. Update compiler/front/msgs.nim 

    Co-authored-by: zerbina <100542850+zerbina@users.noreply.github.com>
    @bung87 @zerbina
    bung87 and zerbina authored Oct 5, 2023
    Configuration menu
    Copy the full SHA
    3be98f4 View commit details
    Browse the repository at this point in the history