Skip to content

Fail if there are 26 letters case sensitive#236

Merged
cmcaine merged 1 commit intoexercism:masterfrom
Wikunia:test-cases-pangram
May 25, 2020
Merged

Fail if there are 26 letters case sensitive#236
cmcaine merged 1 commit intoexercism:masterfrom
Wikunia:test-cases-pangram

Conversation

@Wikunia
Copy link
Copy Markdown
Contributor

@Wikunia Wikunia commented May 25, 2020

I received one solution where it was only checked whether there are exactly 26 letters (case sensitive)
This would result in that a...m A...M is a pangram for example.
Also fails if there are more than 26 letters when counting i.e o and O as two different ones.

@Wikunia
Copy link
Copy Markdown
Contributor Author

Wikunia commented May 25, 2020

Oh I think I should update it here:
https://github.com/exercism/problem-specifications
Correct?

@Wikunia Wikunia closed this May 25, 2020
@cmcaine cmcaine reopened this May 25, 2020
@cmcaine cmcaine merged commit c558904 into exercism:master May 25, 2020
@cmcaine
Copy link
Copy Markdown
Contributor

cmcaine commented May 25, 2020

Thanks!

@cmcaine
Copy link
Copy Markdown
Contributor

cmcaine commented May 25, 2020

Also merged this: exercism/problem-specifications#1658

But we don't have a script to automatically generate the tests from that (for this exercise), so they also have to be added manually.

@cmcaine cmcaine mentioned this pull request May 25, 2020
@Wikunia
Copy link
Copy Markdown
Contributor Author

Wikunia commented May 25, 2020

Ah okay in that case it should be added to the notebook as well, right?

@cmcaine
Copy link
Copy Markdown
Contributor

cmcaine commented May 25, 2020

Yup, also done in #237

@cmcaine
Copy link
Copy Markdown
Contributor

cmcaine commented May 25, 2020

There's a script you can run for it:

julia bin/generate-notebooks.jl

If you speak the magic words, a github action will do it for you, too.

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