Skip to content

completed OOP homework#1

Open
hilwa173 wants to merge 1 commit into
mainfrom
oop-homework
Open

completed OOP homework#1
hilwa173 wants to merge 1 commit into
mainfrom
oop-homework

Conversation

@hilwa173

Copy link
Copy Markdown
Owner

No description provided.

@abiyandargie abiyandargie 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.

I think the code is well!
It is a well-structured.it is well-written and prints out all the information in an organized manner.
keep it up!!

@Benawi

Benawi commented Oct 30, 2023

Copy link
Copy Markdown

Hi @hilwa173 👋

Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

almost

Highlights

  • Invalid code review request
  • You did not commit the required files/classes to the PR

Required Changes ♻️

Check the comments under the review.

Optional suggestions

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.

Cheers, and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
@Benawi

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.

3 participants