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

feat: add remove debugger code action #426

Merged
merged 11 commits into from
May 1, 2024
Merged

feat: add remove debugger code action #426

merged 11 commits into from
May 1, 2024

Commits on Apr 30, 2024

  1. feat: add remove debugger code action 

    A Code Action that removes the following warning checks from Credo:
  - Credo.Check.Warning.Dbg
  - Credo.Check.Warning.IExPry
  - Credo.Check.Warning.IoInspect
  - Credo.Check.Warning.IoPuts
  - Credo.Check.Warning.MixEnv
    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    2345e11 View commit details
    Browse the repository at this point in the history

  2. Refactor to use the AST

    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    d83073b View commit details
    Browse the repository at this point in the history

  3. Fix non-working edge cases. 

    Adds a macro for testing text edits.
Fixes edge-cases when piping debug expr.
Fixes edge-cases when we have a single block with the pipe expr.
    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    c6d5dca View commit details
    Browse the repository at this point in the history

  4. Refactor to smallest ast change possible 

    This refactor takes in only the AST of the debugging node and replaces
it. For IO.inspect/1 and dbg/1 the first argument is preserved.
    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    0f1817b View commit details
    Browse the repository at this point in the history

  5. Fix formatting

    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    13e4fa5 View commit details
    Browse the repository at this point in the history

  6. Refactor the title and some tests

    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    3d136c3 View commit details
    Browse the repository at this point in the history

  7. Move the test helpers to NextLS.Support.Utils

    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    cc28b78 View commit details
    Browse the repository at this point in the history

  8. Add changes to fix some scenarios with pipes.

    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    ccba3ca View commit details
    Browse the repository at this point in the history

  9. Fix formatting and warning

    @NJichev
    NJichev committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    f6f09dc View commit details
    Browse the repository at this point in the history

Commits on May 1, 2024

  1. update credo to origin repo

    @mhanberg
    mhanberg committed May 1, 2024
    Configuration menu
    Copy the full SHA
    31dcea8 View commit details
    Browse the repository at this point in the history

  2. add empty fallback 

    Saw an error in local when i tried to fetch the code actions again
without saving, not really sure if its a good fix but it should stop the
LSP from crashing
    @mhanberg
    mhanberg committed May 1, 2024
    Configuration menu
    Copy the full SHA
    a6b8ddc View commit details
    Browse the repository at this point in the history