Skip to content

UofT-DSI | Python - Assignment 1#1

Open
asnjunior wants to merge 2 commits intomainfrom
assignment-1
Open

UofT-DSI | Python - Assignment 1#1
asnjunior wants to merge 2 commits intomainfrom
assignment-1

Conversation

@asnjunior
Copy link
Copy Markdown
Owner

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

Created code to check if two words are anagrams of each other.

What did you learn from the changes you have made?

Learned how to created functions and methods.

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

NA

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

NA

How were these changes tested?

There were sample inputs/words provided and the expected outcomes. My tests results matched those expected outcomes.

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

NA

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@dwz92 dwz92 left a comment

Choose a reason for hiding this comment

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

You don't have a logic error in your code, but you should be using return to output a value in your function, not print()

@asnjunior
Copy link
Copy Markdown
Owner Author

asnjunior commented Sep 8, 2024

Noted, Emma. Will do that in next assignment. Thanks!

@dwz92
Copy link
Copy Markdown

dwz92 commented Sep 11, 2024

Nope, please fix this error in this assignment so i can pass you.

@asnjunior
Copy link
Copy Markdown
Owner Author

Done. I have made the changes. I hade to do it in GutHub editor though (not through terminal) as I don't have my personal laptop with me right now.

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