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

More informative description of the problem #180

Closed
volodya-lombrozo opened this issue Jun 20, 2023 · 3 comments · Fixed by #181
Closed

More informative description of the problem #180

volodya-lombrozo opened this issue Jun 20, 2023 · 3 comments · Fixed by #181

Comments

@volodya-lombrozo
Copy link
Owner

For now we are printing the next error message:

[ERROR] Test FakeMaven.java doesn't have corresponding production class.
[ERROR]         You can either rename or move the test class /Users/lombrozo/Workspace/EOlang/eo/eo-maven-plugin/src/test/java/org/eolang/maven/FakeMaven.java:
[ERROR]         More about that rule you can read here https://github.com/volodya-lombrozo/jtcop/blob/main/docs/rules/all-have-production-class.md

It is more or less fine, but it has a lack of suggestions what we can do with the problem. I'm sure, that we have to add information how to suppress a rule or how to solve it, at least in a few words. Moreover, we can create a common "Description" class for all rules.

Also, it would be nice to add codename of the rule to the description

@0pdd
Copy link
Collaborator

0pdd commented Jun 20, 2023

@volodya-lombrozo 3 puzzles #182, #183, #184 are still not solved.

@0pdd
Copy link
Collaborator

0pdd commented Jul 8, 2023

@volodya-lombrozo 2 puzzles #182, #183 are still not solved; solved: #184.

@0pdd
Copy link
Collaborator

0pdd commented Jul 9, 2023

@volodya-lombrozo the puzzle #183 is still not solved; solved: #182, #184.

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 a pull request may close this issue.

2 participants