Skip to content

Project ExerciseTracker finished#85

Open
Niasua wants to merge 31 commits intothe-csharp-academy:mainfrom
Niasua:main
Open

Project ExerciseTracker finished#85
Niasua wants to merge 31 commits intothe-csharp-academy:mainfrom
Niasua:main

Conversation

@Niasua
Copy link
Copy Markdown

@Niasua Niasua commented Aug 6, 2025

The changes are in the main and dapper-version branches.

Copy link
Copy Markdown

@chrisjamiecarter chrisjamiecarter left a comment

Choose a reason for hiding this comment

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

Hey @Niasua 👋,


🔴 Project Submission

💡 Please place all your code inside a single top-level folder named: [ProjectName].[UserName].

👨‍🏫 Example: Phonebook.Niasua

📁 CodeReviews.Console. ⬅️ Forked repo
 ┣ 📄 .codacy.yml
 ┣ 📄 .gitignore
 ┣ 📁 Project.Username ⬅️ Top-level folder
 ┃  ┗ 📄 Project.sln
 ┃  ┗ 📄 README.md
 ┃  ┗ 📁 Project.Type

💭 This helps me clearly identify your work and confirms you can work within community repository guidelines.


🔴 Do Not Delete Other Students’ Code

💡 I noticed you removed other student’s project submission folders and code:

Image

❗ This project contains work from multiple students. Each student’s folder represents their own submission. Removing someone else’s folder or files is considered overreach.

🔧 Restore any deleted folders/files belonging to other students. Only modify or remove files inside your own project submission folder.

💭 This is a shared, community repository. We used to merge all other students code into the repository, to preserve everyone’s work for fairness and learning purposes. In real-world collaborative projects, deleting someone else’s code without approval can cause data loss and break builds.


🔧 Next Steps

  • Please fix any 🔴 items, as they block approval.
  • Commit and push your changes, then leave me a comment when done.
  • Feel free to reach out if you have any questions or want to discuss further 🆘.

Thanks,
@chrisjamiecarter 👍

@Niasua
Copy link
Copy Markdown
Author

Niasua commented Aug 12, 2025

Thanks for the review. Sorry about deleting other students' files. I didn't mean it; I thought I should leave only my files when submitting my submission. Thanks for the clarification.

Copy link
Copy Markdown

@chrisjamiecarter chrisjamiecarter left a comment

Choose a reason for hiding this comment

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

Hey @Niasua 👋,

Excellent work on your Exercise Tracker project submission 🎉!

I have performed a peer review. Review/ignore any comments as you wish.


🟢 Requirements

⭐ You have fulfilled all of the project requirements!


🟠 Project Submission

💡 Close! I think you have confused your project submission/top-level folder, with your solution project folder.

Image Image

I will go ahead and mark as approved, keep up the excellent work on the next projects! 😊

Best regards,
@chrisjamiecarter 👍

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

💡 Redundant/dead code should be removed.

Image

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