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.

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

fix(cli): account for parse errors being of string type #444

Merged
merged 3 commits into from
Feb 22, 2025

Conversation

joshzcold
Copy link
Contributor

  • regex parse the incoming string message for its parts.

    Change the case to skip the check on unresolved class to "includes" instead of starts with

Fixes

Proposed Changes

  • cli account for the string errors coming from CodeNarc Server

- regex parse the incoming string message for its parts.

  Change the case to skip the check on unresolved class to "includes"
  instead of starts with
@nvuillam
Copy link
Owner

@joshzcold many thanks for your PR :)

But I can't merge it while there are crashing test methods, please can you solve them then mention me again ?

@joshzcold
Copy link
Contributor Author

@nvuillam now ready 👍

@nvuillam
Copy link
Owner

@joshzcold great ! :)

@nvuillam nvuillam merged commit 3a0509c into nvuillam:main Feb 22, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants