Skip to content

updated task#2587

Open
Seekerfn wants to merge 3 commits into
mate-academy:masterfrom
Seekerfn:my-branch
Open

updated task#2587
Seekerfn wants to merge 3 commits into
mate-academy:masterfrom
Seekerfn:my-branch

Conversation

@Seekerfn

Copy link
Copy Markdown

No description provided.

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, the pull request review was unsuccessful for the following reasons:

Warning

There are no files to review. Please make sure that you have added files to the pull request. Some files and directories may be ignored from the review or failed to load.
[CODE: NFR]

Additional info:
Ignored files list:

  • src/main/java/core/basesyntax/Bulldozer.java
  • src/main/java/core/basesyntax/Excavator.java
  • src/main/java/core/basesyntax/Machine.java
  • src/main/java/core/basesyntax/Truck.java

If the issue is within your control, please address it.
The next review attempt will automatically launch in about 1 hour.
If you believe this is an error or need assistance, please contact the support team.

@Seekerfn Seekerfn closed this May 11, 2026

@Seekerfn Seekerfn left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rewiew

@Seekerfn Seekerfn reopened this May 11, 2026
@Seekerfn

Copy link
Copy Markdown
Author

@mateacademy-ai-mentor

@Seekerfn

Copy link
Copy Markdown
Author

@mateacademy-ai-mentor check my request, please

@Seekerfn Seekerfn left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check next

@Seekerfn Seekerfn left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1

@Seekerfn
Seekerfn marked this pull request as draft May 11, 2026 14:57
@Seekerfn
Seekerfn marked this pull request as ready for review May 11, 2026 14:58
@Seekerfn
Seekerfn marked this pull request as draft May 11, 2026 15:08
@Seekerfn
Seekerfn marked this pull request as ready for review May 11, 2026 15:08

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary


✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨

Footnotes

  1. Rate AI review example

machine.doWork();
machine.stopWork();
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: 'stoped' should be 'stopped'

machine.doWork();
machine.stopWork();
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: 'stoped' should be 'stopped'

machine.doWork();
machine.stopWork();
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: 'stoped' should be 'stopped'


@Override
public void stopWork() {
System.out.println("Bulldozer stoped");

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: 'stoped' should be 'stopped'. The message needs to be accurate and professional.


@Override
public void stopWork() {
System.out.println("Bulldozer stoped");

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: 'stoped' should be 'stopped'. Ensure consistency across all machine classes.


@Override
public void stopWork() {
System.out.println("Excavator stoped");

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: 'stoped' should be 'stopped'

public void doWork() {
System.out.println("Excavator started work");
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Redundant empty line between methods should be removed per checklist guidelines


@Override
public void stopWork() {
System.out.println("Truck stoped");

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: 'stoped' should be 'stopped' - the message should be accurate and professional

@@ -0,0 +1,13 @@
package core.basesyntax;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Redundant empty line after package declaration - remove per checklist guidelines

public void doWork() {
System.out.println("Truck started work");
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Redundant empty line between methods - remove per checklist guidelines

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