[Cool-Katt]

No description provided.

[Cool-Katt]

Okay, so
This is a port of the fourth exercise in the 'resistor colours' series, taken pretty much as is from the Python track.
Discussion can be found here

---

After many attempts, every check except for the links passes. I don't think I can do much about the links in this PR and it's out of scope so it should be fine™️

[glennj]

Resistors are nightmare fuel for the colourblind.

[kotp]

Approving, but provided suggestion that could be put in place, in agreement with @IsaacG .

[zeektvbox01-hub]

Try to add a mention of decimal values

[glennj]

@Cool-Katt where do you stand with this? Just a couple of suggested changes. The link checker stuff will probably self-resolve if you rebase.

[Cool-Katt]

@glennj
Fuck, I'm sorry, I put this stuff on the back-burner and I wanted to take a deeper look at how translations work and I ended up just forgetting....

I'm good with all the suggestions, and I don't really care about the link checker, so If you want to merge this i'm good.

[glennj]

Well, there's the style guide bully, and I think the i18n error is occurring org-wide.

[Cool-Katt]

The way I understood, the i18n error would resolve by putting a ready-to-translate label on this pr when it's ready to translate, hopefully? Do we do that for problem-spec prs too, is it for every pr from now on?

[glennj]

Every PR that touches docs.

[exercism-i18n]

This PR has been translated 🚀

[glennj]

@Cool-Katt the CI failure is not a link checker, its the markdown linter. It will keep failing for subsequent PRs until fixed.

https://github.com/exercism/problem-specifications/actions/runs/38957960230/job/117165491655?pr=2682#step:6:229

[IsaacG]

@Cool-Katt the CI failure is not a link checker, its the markdown linter. It will keep failing for subsequent PRs until fixed.

https://github.com/exercism/problem-specifications/actions/runs/38957960230/job/117165491655?pr=2682#step:6:229

I added "suggestions" to fix the linter

[github-actions]

This push changed English, so ready-to-translate has come off and any queued translation is withdrawn.

A maintainer: please re-apply ready-to-translate once the copy is final. That queues the translation at the PR's head commit.

[Cool-Katt]

Okay, @IsaacG thanks for the suggestions. I ended up having to rerun the lint & format properly (so having to turn my computer on just for that, annoying) but it's now fine, except for the translation.

@glennj I won't re-apply the ready-to-translate label immediately, just in case we need any last 'last minute' changes, so let's let it sit for the weekend and merge next week, if noone has anything to say. Thanks for your help.

[IsaacG]

We have 6 approvals and the other checks pass. I'll use some of my monthly Exercism donations to fund another translation and get that done 😁

[exercism-i18n]

This PR has been translated 🚀