Skip to content

Tests check vowels#691

Merged
raklaptudirm merged 3 commits intoTheAlgorithms:masterfrom
charliejmoore:tests-check-vowels
Sep 20, 2021
Merged

Tests check vowels#691
raklaptudirm merged 3 commits intoTheAlgorithms:masterfrom
charliejmoore:tests-check-vowels

Conversation

@charliejmoore
Copy link
Copy Markdown
Contributor

@charliejmoore charliejmoore commented Sep 19, 2021

Welcome to JavaScript community

Open in Gitpod know more

Describe your change:

Adds additional tests to CheckVowels to cover additional inputs and edge cases.

*[x] Add tests

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new JavaScript files are placed inside an existing directory.
  • All filenames should use the UpperCamelCase (PascalCase) style. There should be no spaces in filenames.
    Example:UserProfile.js is allowed but userprofile.js,Userprofile.js,user-Profile.js,userProfile.js are not

@raklaptudirm raklaptudirm merged commit 3c90d5d into TheAlgorithms:master Sep 20, 2021
@charliejmoore charliejmoore deleted the tests-check-vowels branch September 22, 2021 01:19
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