Skip to content

docs(sector): show the current textile country key in the example - #175

Merged
LKSNDRTMLKV merged 1 commit into
mainfrom
docs/stale-sector-data-example
Aug 22, 2026
Merged

docs(sector): show the current textile country key in the example#175
LKSNDRTMLKV merged 1 commit into
mainfrom
docs/stale-sector-data-example

Conversation

@LKSNDRTMLKV

Copy link
Copy Markdown
Member

SectorData's doc comment showed a textile example carrying
"countryOfManufacturing" directly above a struct that requires
countryOfOrigin — the old key from before the cross-sector rename.

Named in #94 as still visible in the crate's own docs. It is the smallest piece
of that issue and unrelated to the rest, so it goes on its own rather than
riding along with the compatibility gate in #174.

Nothing reads this string, which is exactly why it drifted: cargo doc, clippy
and the test suite all pass over prose without checking it against the type
beside it.

just lint and the doctests pass.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@LKSNDRTMLKV
LKSNDRTMLKV merged commit bb32557 into main Aug 22, 2026
18 checks passed
@LKSNDRTMLKV
LKSNDRTMLKV deleted the docs/stale-sector-data-example branch August 22, 2026 20:57
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