Skip to content

complete#1

Open
SilviMargarian wants to merge 1 commit into
mainfrom
assignment
Open

complete#1
SilviMargarian wants to merge 1 commit into
mainfrom
assignment

Conversation

@SilviMargarian

@SilviMargarian SilviMargarian commented Mar 13, 2026

Copy link
Copy Markdown
Owner

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

I completed the code to check for anagrams when given two strings and a boolean.

What did you learn from the changes you have made?

I learned how to write nested functions.

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

No, I wasn't thinking of another approach.

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

I didn't experience any challenges in this assignment.

How were these changes tested?

By trying examples (i.e., giving two strings and boolean) and checking if it would return the correct response.

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

Not applicable.

Checklist

  • I can confirm that my changes are working as intended

@juliagallucci juliagallucci left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good! One note for improvement would be to simplify your conditional logic. Since word 1 == word 2 is already a Boolean, you do not need to explicitly return True or False!

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