Skip to content

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

Merged
IsaacG merged 3 commits into
exercism:mainfrom
IsaacG:freelance.docs
Oct 30, 2026
Merged

IsaacG merged 3 commits into
exercism:mainfrom
IsaacG:freelance.docs

Conversation

@IsaacG

@IsaacG IsaacG commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

@IsaacG

IsaacG commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

+cc @exercism/cpp

@siebenschlaefer

Copy link
Copy Markdown
Contributor

I often mix up the terms "argument" and "parameter" myself, but should this new text use "arguments"?
Because a function has parameters but takes arguments.

@IsaacG

IsaacG commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

should this new text use "arguments"?

Hrm. Probably. I could make that change across the various tracks.

@IsaacG

IsaacG commented Oct 29, 2026

Copy link
Copy Markdown
Member Author

What about lines 17 and 30?

Implement a function called daily_rate to calculate the daily rate given an hourly rate as a parameter.

It should accept two parameters: the original price and the discount rate in percent.

Should those be changed, too?

@siebenschlaefer

Copy link
Copy Markdown
Contributor

Good catch. IMHO yes.

angelikatyborska pushed a commit to exercism/elixir that referenced this pull request Oct 30, 2026

@vaeng vaeng left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are probably more of these on the track. I remember I had a discussion about this with Siebenschläfer, so we might have correct some of those.

@IsaacG IsaacG added x:module/concept-exercise Work on Concept Exercises x:type/docs Work on Documentation x:size/tiny Tiny amount of work x:rep/tiny Tiny amount of reputation ready-to-translate The English in this PR is final; queue its translations labels Oct 30, 2026
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Oct 30, 2026
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Oct 30, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

@IsaacG
IsaacG merged commit 8d139cb into exercism:main Oct 30, 2026
27 of 28 checks passed
@IsaacG
IsaacG deleted the freelance.docs branch October 30, 2026 15:48
keiravillekode pushed a commit to exercism/x86-64-assembly that referenced this pull request Oct 30, 2026
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 x:module/concept-exercise Work on Concept Exercises x:rep/tiny Tiny amount of reputation x:size/tiny Tiny amount of work x:type/docs Work on Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants