Skip to content

Completed Assignment 1 - Anagram Checker#266

Closed
YuouSheng wants to merge 1 commit intoUofT-DSI:mainfrom
YuouSheng:assignment-1
Closed

Completed Assignment 1 - Anagram Checker#266
YuouSheng wants to merge 1 commit intoUofT-DSI:mainfrom
YuouSheng:assignment-1

Conversation

@YuouSheng
Copy link
Copy Markdown

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

What did you learn from the changes you have made?

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

How were these changes tested?

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

@tianyi21
Copy link
Copy Markdown
Collaborator

tianyi21 commented Oct 21, 2025

Hi @YuouSheng . When submitting your assignment, please follow the submission guideline to create a PR from your assignment-1 branch to your forked main branch instead of the DSI:main branch. This PR will be closed shortly.

@tianyi21 tianyi21 closed this Oct 21, 2025
@YuouSheng
Copy link
Copy Markdown
Author

YuouSheng commented Oct 21, 2025 via email

@tianyi21
Copy link
Copy Markdown
Collaborator

Hi Tianyi, I've submitted another one. Could you have a look at this one to see if it's correct? Thank you! YuouSheng#1

------------------- Yuou Sheng Ph.D. Student Language and Literacies Education OISE, University of Toronto
________________________________ From: Tianyi Liu @.> Sent: October 20, 2025 22:30 To: UofT-DSI/python @.> Cc: Yuou Sheng @.>; Mention @.> Subject: Re: [UofT-DSI/python] Completed Assignment 1 - Anagram Checker (PR #266) You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification Closed #266<#266>. — Reply to this email directly, view it on GitHub<#266 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BX6RC3M2BARXKEZLJUFVX6L3YWLE7AVCNFSM6AAAAACJWYTWXSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMRQGM4TEOBWGYZTSOI. You are receiving this because you were mentioned.Message ID: @.***>

Yep, that the correct one and we have confirmed earlier 😃. I was in the process of closing PRs people accidentally created to the DSI repo.

@YuouSheng
Copy link
Copy Markdown
Author

YuouSheng commented Oct 21, 2025 via email

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