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

Remove unwanted characters from error message #577

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

Bananeweizen
Copy link
Contributor

What's changed?

The error message for resources with parse errors.

What's your motivation?

The error message currently says "There were problems parsing 7 + sources:".

Looking at the git history the plus sign in the output is probably from a string concatenation that made it into the string literal by accident.

Checklist

n.a.

The error message currently says "There were problems parsing 7 +
sources:".

Looking at the git history the plus sign in the output is probably from
a string concatenation that made it into the string literal by accident.
@timtebeek timtebeek self-assigned this Jun 6, 2023
@timtebeek timtebeek added the bug Something isn't working label Jun 6, 2023
@timtebeek timtebeek self-requested a review June 6, 2023 19:53
@timtebeek timtebeek assigned Bananeweizen and unassigned timtebeek Jun 6, 2023
@timtebeek
Copy link
Contributor

I'm going to agree with you here that the message is sub optimal. Ideally I think we additionally run Slf4jBestPractices to parameterize the logging statement instead of using String concatenation; should be an easy follow up after this is merged. Thanks for pointing it out!

@timtebeek timtebeek merged commit a539d89 into openrewrite:main Jun 6, 2023
@Bananeweizen Bananeweizen deleted the fix_error_message branch June 7, 2023 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants