Skip to content

Finished Project#386

Open
ktawiah wants to merge 5 commits intoTheCSharpAcademy:masterfrom
ktawiah:master
Open

Finished Project#386
ktawiah wants to merge 5 commits intoTheCSharpAcademy:masterfrom
ktawiah:master

Conversation

@ktawiah
Copy link

@ktawiah ktawiah commented Jan 15, 2026

No description provided.

@chrisjamiecarter chrisjamiecarter self-assigned this Feb 1, 2026
Copy link
Collaborator

@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 @ktawiah 👋,

Excellent work on your Habit Tracker project submission 🎉!

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


🔴 Missing Requirement

❗ As per the project requirements:
🐞 Your project needs to contain a Read Me file where you'll explain how your app works and tell a little bit about your thought progress.


🟠 Project Submission

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

👨‍🏫 Example: HabitTracker.ktawiah

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

🟠 Style Guide Compliance

💡 Remember that when you have finished a project, give it a polish pass. Refer to the Academy's code-conventions page.

💭 This is important as good code conventions/styling ensures consistency, makes your code easier for others to read and maintain, and improves your chances of getting a pull request reviewed and approved.


🔧 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 👍

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