flatten-array Add additional test cases#1953
Merged
BethanyG merged 2 commits intoexercism:mainfrom Feb 7, 2022
Merged
Conversation
BethanyG
approved these changes
Feb 7, 2022
Contributor
|
Three suggestions: (Disclaimer: I'm not a native English speaker.) |
Member
Which makes you much better at catching typos! |
BethanyG
reviewed
Feb 7, 2022
BethanyG
reviewed
Feb 7, 2022
BethanyG
requested changes
Feb 7, 2022
Member
BethanyG
left a comment
BethanyG
left a comment
There was a problem hiding this comment.
Put in the suggestions for typo correction caught by @siebenschlaefer.
kwchang0831
force-pushed
the
1951-proposal2-add-test-cases
branch
from
1430783 to
8944444
Compare
kwchang0831
force-pushed
the
1951-proposal2-add-test-cases
branch
from
8944444 to
737b36e
Compare
Contributor
Author
|
Really appreciate both your help. I have updated the typo and suggested changes. |
Member
|
Paging @exercism/reviewers for additional reviews/approval. 😄 |
junedev
approved these changes
Feb 7, 2022
neenjaw
approved these changes
Feb 7, 2022
petertseng
approved these changes
Feb 7, 2022
Co-authored-by: Peter Tseng <petertseng@users.noreply.github.com>
ErikSchierboom
added a commit
to ErikSchierboom/problem-specifications
that referenced
this pull request
Feb 11, 2022
* Format using prettier (exercism#1917) Format using prettier * updated description of anagrams exercise (exercism#1928) * updated description of anagrams * changed anagram description to be one-sentence-per-line * updated description of anagrams to use sets * Update Licence Give a look at the discussion in BR exercism#1930 * rational-numbers: test to reduce abs value (exercism#1938) * Change saddle point references to row, column (exercism#1948) * word-search: Add test case * Update exercises/word-search/canonical-data.json Agreed. Co-authored-by: Erik Schierboom <erik_schierboom@hotmail.com> * meetup: improve descriptions by saying why each case is tested (exercism#1919) descriptions show whether a date is the first, last, or an arbitrary middle date of the week. This helps understand why certain cases are selected. Closes exercism#974 * word-search: Add cases checking for concatenation and wrapping The author of this commit thinks that concatenation is highly unlikely, but the wrapping might be useful to check in languages that allow negative indices. * `flatten-array` Add additional test cases (exercism#1953) * Add additional test cases to flatten-array * Update exercises/flatten-array/canonical-data.json Co-authored-by: Peter Tseng <petertseng@users.noreply.github.com> Co-authored-by: BethanyG <BethanyG@users.noreply.github.com> Co-authored-by: Peter Tseng <petertseng@users.noreply.github.com> * Fix bowling game copy (exercism#1955) Fixes exercism#1954 * Add action to format code (exercism#1941) * build(deps): bump DavidAnson/markdownlint-cli2-action (exercism#1952) Bumps [DavidAnson/markdownlint-cli2-action](https://github.com/DavidAnson/markdownlint-cli2-action) from 5.0.0 to 5.1.0. - [Release notes](https://github.com/DavidAnson/markdownlint-cli2-action/releases) - [Commits](DavidAnson/markdownlint-cli2-action@b3c3b40...744f913) --- updated-dependencies: - dependency-name: DavidAnson/markdownlint-cli2-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Reduced rational nr. should be in standard form. (exercism#1958) * Reduced rational should be in standard form. The current instructors fail to mention that a reduced rational number should always be rendered in standard form (without any negative value at the denominator). * remove superflous blank lines; fix wording * scale-generator: use flat and sharp symbols (exercism#1942) * Update configlet part in README (exercism#1949) Co-authored-by: ee7 <45465154+ee7@users.noreply.github.com> * phone number: only one problem per test input (exercism#1959) * [Phone Number] Only one problem per test input Because the area code is not allowed to start with 0 or 1, inputs designed to elicit other errors should not use area codes that start with either of those digits. * Respect immutability * Correct field name: s/comment/comments/ * Comments should contain a list. * Allow prettier to improve comments * book-store: reorder keys * darts: reorder keys * grade-school: reorder keys * hamming: reorder keys * high-scores: reorder keys * largest-series-product: reorder keys * list-ops: reorder keys * luhn: reorder keys * triangle: reorder keys * scale-generator: reorder keys * saddle-points: reorder keys * diffie-hellman: reorder keys * collatz-conjecture: reorder keys * anagram: reorder keys * accumulate: reorder keys * Add CI script to check correct order of keys Co-authored-by: Bart Massey <bart.massey@gmail.com> Co-authored-by: y8l <72172663+yusufadell@users.noreply.github.com> Co-authored-by: Ivan Ivanov <mr.mstranger@gmail.com> Co-authored-by: Damian C. Rossney <dcr8898@users.noreply.github.com> Co-authored-by: mariohuq <mariohuq@github.com> Co-authored-by: mariohuq <mario.huq@gmail.com> Co-authored-by: Peter Tseng <petertseng@users.noreply.github.com> Co-authored-by: Peter Tseng <pht24@cornell.edu> Co-authored-by: AH WEI <88667293+kwchang0831@users.noreply.github.com> Co-authored-by: BethanyG <BethanyG@users.noreply.github.com> Co-authored-by: Cedd Burge <ceddlyburge@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Davide Alberto Molin <davide@davidemolin.com> Co-authored-by: wolf99 <281700+wolf99@users.noreply.github.com> Co-authored-by: June <12543047+junedev@users.noreply.github.com> Co-authored-by: ee7 <45465154+ee7@users.noreply.github.com> Co-authored-by: Leah Hanson <astrieanna@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
2nd Proposal of #1951
Add additional test cases to test if solution can correctly filter out null values from input list without filter out other non-null values.