Skip to content

Add save the cow exercise - #891

Merged
glennj merged 7 commits into
exercism:mainfrom
resu-xuniL:add-save-the-cow
Sep 23, 2026
Merged

glennj merged 7 commits into
exercism:mainfrom
resu-xuniL:add-save-the-cow

Conversation

@resu-xuniL

Copy link
Copy Markdown
Contributor

Hello!
Here is save the cow exercise for validation:

  • Generated exercice files with configlet create
  • Created template.j2 .bats file and example.sh
  • Set the difficulty rank to 3
  • Added GitHub handle to config.json

Is dificulty rank 3 OK?


Reviewer Resources:

Track Policies

Comment thread exercises/practice/save-the-cow/save_the_cow.bats Outdated
Comment thread exercises/practice/save-the-cow/save_the_cow.bats Outdated
Comment thread exercises/practice/save-the-cow/.meta/example.sh Outdated
@glennj

glennj commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

closing and reopening just to satisfy the i18n CI check

@glennj glennj closed this Sep 23, 2026
@glennj glennj reopened this Sep 23, 2026
@glennj

glennj commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

My only other thought: is this a "high-easy" or "low-medium" exercise? Should the difficulty be 3 or 4. It's a real nitpick, but should this be displayed as "easy" or "medium" on the website.

@exercism/bash @resu-xuniL what do you think?

@glennj

glennj commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Once we're completely ready to merge, add the "ready-to-translate" label as per exercism/problem-specifications#2686

@IsaacG

IsaacG commented Sep 23, 2026

Copy link
Copy Markdown
Member

My only other thought: is this a "high-easy" or "low-medium" exercise? Should the difficulty be 3 or 4. It's a real nitpick, but should this be displayed as "easy" or "medium" on the website.

@exercism/bash @resu-xuniL what do you think?

I'd bump it to medium. It's got enough going on that it isn't quite easy.

@resu-xuniL

Copy link
Copy Markdown
Contributor Author

Once we're completely ready to merge, add the "ready-to-translate" label as per exercism/problem-specifications#2686

Looking how to do this...

@glennj glennj added the ready-to-translate The English in this PR is final; queue its translations label Sep 23, 2026
@resu-xuniL

Copy link
Copy Markdown
Contributor Author

Thanks @glennj

@exercism-i18n

exercism-i18n Bot commented Sep 23, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

Comment thread exercises/practice/save-the-cow/save_the_cow.bats Outdated
Comment thread exercises/practice/save-the-cow/save_the_cow.bats Outdated
Comment thread config.json Outdated
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Sep 23, 2026
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@glennj
glennj merged commit fce8a7f into exercism:main Sep 23, 2026
10 of 12 checks passed
@resu-xuniL
resu-xuniL deleted the add-save-the-cow branch September 24, 2026 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-translate The English in this PR is final; queue its translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants