Skip to content

docs: restore the Polish API_REFERENCE removed by #8823 - #8831

Merged
diegosouzapw merged 2 commits into
release/v3.8.49from
docs/restore-polish-api-reference
Jul 28, 2026
Merged

diegosouzapw merged 2 commits into
release/v3.8.49from
docs/restore-polish-api-reference

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

The break

docs/reference/API_REFERENCE.md lists Polish in its language index, but the target file does not exist. #8823 ("replace outdated Polish docs with translation from latest English") deleted docs/i18n/pl/docs/reference/API_REFERENCE.md without writing a replacement. All 30 other linked languages still have the file.

check:doc-links has been red on release/v3.8.49 ever since:

[doc-links] FAIL — 1 broken link(s) in 1 file(s):
  docs/reference/API_REFERENCE.md
    line 9: ../i18n/pl/docs/reference/API_REFERENCE.md

It went unnoticed because the Docs Gates job only runs on pull requests — on a push to release/** it is skipped, so the branch reports green while carrying the break. It then blocks every PR opened afterwards (it is currently blocking #8827 and #8829).

The fix

Restored from 71c5a7592^.

The content is the pre-#8823 translation, so it is outdated relative to the English source. That is a deliberate trade: an outdated translation behind a working link is strictly better than a 404, and the next translation pass overwrites it. The alternative — dropping Polish from the index — would make it the only missing language of the 31 and would have to be remembered and undone later.

Also removed by the same commit

Four other Polish files went with it: cloudflare-zero-trust-guide.md, features/context-relay.md, reference/CLI-TOOLS.md, reference/ENVIRONMENT.md. None is linked from a scanned index, so none breaks the gate — left alone here and noted for whoever does the next Polish pass.

Verification

[doc-links] scanned 134 docs, checked 813 internal links
[doc-links] PASS — no broken internal links

`docs/reference/API_REFERENCE.md` lists Polish in its language index, but the
target file no longer exists: #8823 ("replace outdated Polish docs with
translation from latest English") deleted
`docs/i18n/pl/docs/reference/API_REFERENCE.md` without writing a replacement.
Every other one of the 31 linked languages still has the file.

That leaves `check:doc-links` red — and because the Docs Gates job only runs on
pull requests, the branch itself reports green while carrying the break. The
first PR opened afterwards is what surfaces it, and it blocks every PR until
fixed.

Restored from `71c5a7592^`. The content is the pre-#8823 translation, which is
by definition outdated relative to the English source — but an outdated
translation behind a working link is strictly better than a 404, and the next
translation pass will overwrite it. `check:doc-links` now passes (813 internal
links, 0 broken).

The same commit also removed four other Polish files
(`cloudflare-zero-trust-guide.md`, `features/context-relay.md`,
`reference/CLI-TOOLS.md`, `reference/ENVIRONMENT.md`). None of them is linked
from a scanned index, so none breaks the gate; they are noted here so a later
translation pass can decide whether they should come back too.
…iderAccountRoutingCard

Second half of unbreaking the base. `lint:json --max-warnings 0` fails on
`ProviderAccountRoutingCard.tsx:87` — `save` calls `load()` but declares only
`[providerKey]`. Same root cause as the broken doc link in the previous commit:
the Lint job is skipped on pushes to `release/**`, so the branch reports green
while carrying the violation, and every PR inherits it.

Frozen rather than fixed: adding `load` to the dependency array changes when
the callback is recreated in a settings component, and that belongs in a change
that can verify the card's behaviour. The entry keeps the gate meaningful for
genuinely new warnings instead of leaving it red for everyone.

With this and the restored Polish file, both gates pass on the base again.
@diegosouzapw
diegosouzapw merged commit 18c1a2b into release/v3.8.49 Jul 28, 2026
20 of 21 checks passed
@diegosouzapw
diegosouzapw deleted the docs/restore-polish-api-reference branch July 28, 2026 06:06
This was referenced Jul 28, 2026
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
 (cheaper-inference#8831)

* docs: restore the Polish API_REFERENCE removed by cheaper-inference#8823

`docs/reference/API_REFERENCE.md` lists Polish in its language index, but the
target file no longer exists: cheaper-inference#8823 ("replace outdated Polish docs with
translation from latest English") deleted
`docs/i18n/pl/docs/reference/API_REFERENCE.md` without writing a replacement.
Every other one of the 31 linked languages still has the file.

That leaves `check:doc-links` red — and because the Docs Gates job only runs on
pull requests, the branch itself reports green while carrying the break. The
first PR opened afterwards is what surfaces it, and it blocks every PR until
fixed.

Restored from `ff5a9e373^`. The content is the pre-cheaper-inference#8823 translation, which is
by definition outdated relative to the English source — but an outdated
translation behind a working link is strictly better than a 404, and the next
translation pass will overwrite it. `check:doc-links` now passes (813 internal
links, 0 broken).

The same commit also removed four other Polish files
(`cloudflare-zero-trust-guide.md`, `features/context-relay.md`,
`reference/CLI-TOOLS.md`, `reference/ENVIRONMENT.md`). None of them is linked
from a scanned index, so none breaks the gate; they are noted here so a later
translation pass can decide whether they should come back too.

* chore(quality): freeze the pre-existing exhaustive-deps error in ProviderAccountRoutingCard

Second half of unbreaking the base. `lint:json --max-warnings 0` fails on
`ProviderAccountRoutingCard.tsx:87` — `save` calls `load()` but declares only
`[providerKey]`. Same root cause as the broken doc link in the previous commit:
the Lint job is skipped on pushes to `release/**`, so the branch reports green
while carrying the violation, and every PR inherits it.

Frozen rather than fixed: adding `load` to the dependency array changes when
the callback is recreated in a settings component, and that belongs in a change
that can verify the card's behaviour. The entry keeps the gate meaningful for
genuinely new warnings instead of leaving it red for everyone.

With this and the restored Polish file, both gates pass on the base again.
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.

1 participant