Skip to content

Conversation

@justin212407
Copy link
Member

Week 3 DMP 2025 Blog for Masonry

@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@github-actions
Copy link

github-actions bot commented Jul 1, 2025

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

Copy link
Contributor

@pikurasa pikurasa left a comment

Choose a reason for hiding this comment

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

Please address these issues.

Also: https://www.sugarlabs.org/news/all/2025-06-15-dmp-25-justin212407-week02#week-2-progress-report-by-safwan-sayeed says "Progress report by Safwan ..."

Please fix that in another PR.

Image:
Screenshot from 2025-07-01 10-31-31

Also, I'm a bit unsure, but you may have three "Week 2s". I know you misnamed one in this PR, which accounts for one of them. Please test after you've made the change by going to "news" and searching for your name. If there are still duplicates, please investigate. Perhaps there is a redundant file.

@@ -0,0 +1,100 @@
---
title: "DMP’25 Week 02 Update by Justin Charles"
Copy link
Contributor

Choose a reason for hiding this comment

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

Title says "Week 02"

Did you try your code, because the errors are obvious when you test it out. For example:

Screenshot from 2025-07-01 10-26-49


**Project:** Music Blocks 4 Maosonry
**Mentors:** [Anindya Kundu](https://github.com/meganindya/), [Sumit Srivastava](https://github.com/sum2it)
**Assisting Mentors:** [Devin Ullibari](https://github.com/pikurasa/), [Walter Bender](https://github.com/walterbender)
Copy link
Contributor

Choose a reason for hiding this comment

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

My name is "Devin Ulibarri" (Only one "L")

@github-actions
Copy link

github-actions bot commented Jul 1, 2025

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@justin212407
Copy link
Member Author

@pikurasa made all of the requested changes... sorry for the errors created these blogs while travelling so i overlooked a few of the mistakes i made

@pikurasa pikurasa merged commit 72e874e into sugarlabs:main Jul 8, 2025
3 checks passed
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