Skip to content

[freelancer-rates] Tidy up instructions, syncing wording across cpp, elixir and x86-64-assembly - #1654

Merged
angelikatyborska merged 1 commit into
exercism:mainfrom
IsaacG:freelance.docs
Oct 2, 2026
Merged

angelikatyborska merged 1 commit into
exercism:mainfrom
IsaacG:freelance.docs

Conversation

@IsaacG

@IsaacG IsaacG commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Thank you for contributing to exercism/elixir 💜 🎉. This is an automated PR comment 🤖 for the maintainers of this repository that helps with the PR review process. You can safely ignore it and wait for a maintainer to review your changes.

Based on the files changed in this PR, it would be good to pay attention to the following details when reviewing the PR:

  • General steps

    • 🏆 Does this PR need to receive a label with a reputation modifier (x:size/{tiny,small,medium,large,massive})? (A medium reputation amount is awarded by default, see docs)
  • Any exercise changed

    • 👤 Does the author of the PR need to be added as an author or contributor in <exercise>/.meta/config.json (see docs)?
    • 🔬 Do the analyzer and the analyzer comments exist for this exercise? Do they need to be changed?
    • 📜 Does the design file (<exercise>/.meta/design.md) need to be updated to document new implementation decisions?
  • Concept exercise changed

    • 🌲 Do prerequisites and practices in config.json need to be updated?
    • 📖 Does the concept introduction provide all necessary information to solve this exercise?

Automated comment created by PR Commenter 🤖.

@angelikatyborska

Copy link
Copy Markdown
Member

Thanks for the changes. They look good. Looks like there's improved grammar in here 👍 but technically speaking, we don't really need to be learning exercises consistent between tracks, right? When we first wrote them, that was the whole point - each track can write whatever they want and there's no problem specs.

Btw. I didn't look into the i18n things yet, but will that check "i18n completeness" keep failing for all PRs? It looks like it wants us to translate something to many different languages. But I'm not expected to translate to Hungarian and Chinese every time I change something in this repo, am I?

@IsaacG

IsaacG commented Oct 2, 2026

Copy link
Copy Markdown
Member Author

technically speaking, we don't really need to be learning exercises consistent between tracks, right

Absolutely! And these PRs do not make the docs completely consistent. They all each have track specific bits (double vs Double vs float and much more)! However, the prose bits have been improved over time on the various tracks. It reduces maintainer effort and improves doc quality if we do sync improvements across tracks. It's not necessary, but I believe it's strictly better. By keeping the docs consistent-ish, future doc improvements are easier to cross apply, amplifying the impact of changes with reduces efforts.

Do we need to do this? Not at all. Does this improve Exercism? I strongly believe it does!

Btw. I didn't look into the i18n things yet, but will that check "i18n completeness" keep failing for all PRs?

The translations are done via AI. No human work needed. Details here: exercism/problem-specifications#2686

TL;DR: once the PR is approved and ready to ship (no future changes to avoid re-translating), add a PR label ready-to-translate. GitHub CI/CD handles the rest. Wait ~5-10 minutes and translations should be done and the CI check should be green.

@angelikatyborska angelikatyborska added the ready-to-translate The English in this PR is final; queue its translations label Oct 2, 2026
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Oct 2, 2026
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Oct 2, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

@angelikatyborska

Copy link
Copy Markdown
Member

No failures. Cost: $0.0314.

That's impressively low costs 🤔

@angelikatyborska
angelikatyborska merged commit 9fcea7d into exercism:main Oct 2, 2026
15 of 16 checks passed
@angelikatyborska

Copy link
Copy Markdown
Member

Thanks a pretty neat translation process. Thanks for helping out 🌷

@IsaacG

IsaacG commented Oct 2, 2026

Copy link
Copy Markdown
Member Author

No failures. Cost: $0.0314.

That's impressively low costs 🤔

Apparently DeepSeek is far cheaper than US alternatives.

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.

2 participants