Skip to content

Conversation

@Bishoywadea
Copy link
Contributor

Description

Report for week 2.

Changes Made

Added 2025-06-15-gsoc-25-BishoyWadea-week02.md-> blog report

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added/updated tests that prove the effectiveness of these changes.
  • I have updated the documentation to reflect these changes, if applicable.
  • I have followed the project's coding style guidelines.
  • I have addressed the code review feedback from the previous submission, if applicable.

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

@sa-fw-an
Copy link
Member

@Bishoywadea Please Add an Authors file and change the titles of the blog. For documentation refer this

Change this PR and also for week 1 follow the new convention and change that as well

Have a look at my blog for new authors page and make a page somewhat like this

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.

There are a lot of broken links.

How did that happen?

Please fix those, as well as the new author field.

1. **Initial Setup & Core Functionality**
- Added starter files and project structure.
- Implemented basic game manager functionality to handle state, inputs, and equation validation.
- commit: [Initial Commit](https://github.com/Bishoywadea/BrokenCalculator/commit/initial)
Copy link
Contributor

Choose a reason for hiding this comment

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

There are a lot of broken links in your blogpost, such as this one.

Please check all your links before making a PR.

Screenshot from 2025-06-15 08-39-23


---

## Connect with Me
Copy link
Contributor

Choose a reason for hiding this comment

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

Please follow Sawfan's new guidance about creating a profile page for authorship.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done check it now

@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

🎉 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

🎉 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. 👏

@Bishoywadea
Copy link
Contributor Author

@Bishoywadea Please Add an Authors file and change the titles of the blog. For documentation refer this

Change this PR and also for week 1 follow the new convention and change that as well

Have a look at my blog for new authors page and make a page somewhat like this

Done check it now

category: "DEVELOPER NEWS"
date: "2025-06-07"
slug: "2025-06-07-gsoc-25-BishoyWadea-week01"
author: "Bishoy Wadea"
Copy link
Member

Choose a reason for hiding this comment

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

You need to pass this @/constants/MarkdownFiles/authors/bishoy-wadea.md instead of this for author

category: "DEVELOPER NEWS"
date: "2025-06-15"
slug: "gsoc-25-BishoyWadea-week01"
author: "Bishoy Wadea"
Copy link
Member

Choose a reason for hiding this comment

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

Follow the comment above for author.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, sorry, I didn't notice.

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

Comment on lines 112 to 118
## Connect with Me

- GitHub: [@Bishoywadea](https://github.com/Bishoywadea)
- Gmail: [[email protected]](mailto:[email protected])
- LinkedIn: [Bishoy Wadea](https://www.linkedin.com/in/bishoy-wadea-27b016250/)

---
Copy link
Member

Choose a reason for hiding this comment

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

You can remove this section here and in blog 1 as well. It is already there in Authors page

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

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

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.

Thank you for fixing the links.

Please review your links etc. prior to future PRs.

@pikurasa pikurasa merged commit 0b270d4 into sugarlabs:main Jun 16, 2025
3 checks passed
saumyashahi pushed a commit to saumyashahi/www-v2 that referenced this pull request Jun 17, 2025
* second week blog

* fix typo

* add my author file

* update titles  to new conventions

* update author

* remove connect with me
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