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

[FIX] Added edge cases and minor spelling fixes to tests #886

Merged
merged 6 commits into from Dec 6, 2018

Conversation

Projects
None yet
2 participants
@itchylol742
Copy link
Contributor

commented Dec 3, 2018

Description

Added edge cases for compact, countOccurances, filterNonUnique, squareSum, anduniqueSymmetricDifference, and spelling fixes to average and findLastKey.

PR Type

  • Snippets, Tests & Tags (new snippets, updated snippets, re-tagging of snippets, added/updated tests)
  • Scripts & Website & Meta (anything related to files in the scripts folder, how the repository's automated procedures work and the website)
  • Glossary & Secondary Features (anything related to the glossary, such as new or updated terms or other secondary features)
  • General, Typos, Misc. & Meta (everything else, typos, general stuff and meta files in the repository - e.g. the issue template)

Guidelines

  • I have read the guidelines in the CONTRIBUTING document.
  • My PR doesn't include any testlog changes.

itchylol742 added some commits Dec 3, 2018

Merge pull request #2 from 30-seconds/master
update from upstream

@Chalarangelo Chalarangelo merged commit cb60a09 into 30-seconds:master Dec 6, 2018

1 check passed

security/snyk - package.json (Chalarangelo) No manifest changes detected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.