Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

accidental pull request. Please ignore. #16

Closed
wants to merge 6 commits into from
Closed

accidental pull request. Please ignore. #16

wants to merge 6 commits into from

Conversation

pengnam
Copy link

@pengnam pengnam commented Sep 5, 2018

No description provided.

input.txt has comments that indicate that the expected output of a find command with a lowercase form of a keyword is 0 results found. Amend for new lowercase find feature
Finding using a lowercase form of a keyword should be valid and return results
Amend getPersonsWithNameContainingAnyKeyword
…ileForStorage

Both functions are similar. Commit pulls up both functions into setupFileForStorage. Shifts default load showToUser call to processProgramArgs
@pengnam pengnam closed this Sep 5, 2018
@pengnam
Copy link
Author

pengnam commented Sep 5, 2018

Whooops accidental. Please ignore.

@pengnam pengnam changed the title second request accidental pull request. Please ignore. Sep 5, 2018
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.

1 participant