Skip to content

Python Assignment 1#1

Open
JesusSolisR wants to merge 1 commit intomainfrom
assignment-1
Open

Python Assignment 1#1
JesusSolisR wants to merge 1 commit intomainfrom
assignment-1

Conversation

@JesusSolisR
Copy link
Copy Markdown
Owner

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

Adding the required code for the Anagram Checker function and validating test cases

What did you learn from the changes you have made?

Use different python functions, syntax, conditions.

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

Initially I tried to build a function to compare and sort the string using a list of all the letters, until doing a little bit of research found the sort function.

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

None

How were these changes tested?

Running the Jupyter notebook

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

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

@juliagallucci juliagallucci left a comment

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, you don’t need to explicitly use an if/else to return True or False- you can just return the comparison directly.

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