docs: README badges + dash hygiene + drop stale legacy-provider claim#108
Merged
Conversation
Pulls 6 more FCS commits — "Update structs", InstanceContentType update, EurekaState struct addition, TerritoryIntendedUse.Unk65, AgentHudExpFlag tweak, data.yml refresh. All 179 tests pass; no integrity-test pin drift. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… claim
Adds a badge block under the title (CI / release status, GitHub release
+ download counts, NuGet stable + pre-release version + total downloads,
Discord). Mirrors the layout of the Chromatics README so a reader who
knows that project recognises the same set of signals here.
Anti-AI cleanup pass on the prose:
- em dashes in body text and in the Credits bullet list replaced with
regular hyphens-with-spaces
- en dash in "2021–2026" replaced with a regular hyphen
- line 12 corrected: previously claimed LegacySharlayanResources was
available as an opt-in [Obsolete] fallback. Per CLAUDE.md it was
deleted in the 9.0 rebuild; the README now states there is no
fallback path.
No code changes; version bump (9.0.31 → 9.0.32) reflects the user-facing
documentation update.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The Sharlayan distribution channel is NuGet — GitHub release downloads are effectively zero, so the badges added the wrong signal. Keeping the NuGet stable / pre-release / total-downloads + CI / Release workflow status + Discord. Latest GitHub release version stays implied by the NuGet version badges. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two small follow-ups on top of the merged 9.0 work plus an FCS submodule bump.
Badge block (CI / NuGet / Discord)
Adds shields.io badges under the title, mirroring the layout of the Chromatics README:
main)GitHub release download counters were initially included but dropped — Sharlayan ships through NuGet so those counts are always ~0 and would be misleading.
Anti-AI prose pass
2021–2026replaced with a regular hyphen.LegacySharlayanResourcescorrected — previously claimed it was an opt-in[Obsolete]fallback, but perCLAUDE.mdit was deleted in the 9.0 rebuild. README now states there is no fallback path.FCS submodule bump
7891a3d2a → 78d04f840(6 commits —Update structs,InstanceContentTypeupdate,EurekaStatestruct addition,TerritoryIntendedUse.Unk65,AgentHudExpFlagtweak, data.yml refresh).Test plan
dotnet test— 179/179 passingmain🤖 Generated with Claude Code