Skip to content

Tests for check flat case#689

Merged
raklaptudirm merged 6 commits intoTheAlgorithms:masterfrom
charliejmoore:tests-check-flat-case
Sep 19, 2021
Merged

Tests for check flat case#689
raklaptudirm merged 6 commits intoTheAlgorithms:masterfrom
charliejmoore:tests-check-flat-case

Conversation

@charliejmoore
Copy link
Copy Markdown
Contributor

@charliejmoore charliejmoore commented Sep 14, 2021

Welcome to JavaScript community

Open in Gitpod know more

Describe your change:

  • Created a test file and added tests for CheckFlatCase.js
  • Updated the documentation with an explanation of what flat case is
  • Updated names/types to align with js conventions
  • Documentation change?
  • 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
Copy link
Copy Markdown
Member

Please use standard.js to format your code.

@raklaptudirm raklaptudirm added code style issue Failing style checks Reviewed labels Sep 18, 2021
@raklaptudirm raklaptudirm removed the code style issue Failing style checks label Sep 19, 2021
@raklaptudirm raklaptudirm merged commit e40c62d into TheAlgorithms:master Sep 19, 2021
@charliejmoore charliejmoore deleted the tests-check-flat-case branch September 19, 2021 18:50
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