Skip to content

Merge mergeable open PRs from upstream Donkie/SpoolmanDB#1

Merged
sherrmann merged 36 commits into
mainfrom
claude/upstream-pr-review-merge-kobghs
Jul 2, 2026
Merged

Merge mergeable open PRs from upstream Donkie/SpoolmanDB#1
sherrmann merged 36 commits into
mainfrom
claude/upstream-pr-review-merge-kobghs

Conversation

@sherrmann

Copy link
Copy Markdown
Owner

Summary

Reviewed all 39 open PRs on upstream Donkie/SpoolmanDB and merged 35 of them as 36 commits (one commit per PR, original authors credited via --author). Upstream has been inactive since Nov 2025, so our main was exactly even with upstream main — every patch was applied against the same base upstream CI would use.

Compiled database grows from 6,957 → 8,487 entries.

Merged as-is (24 PRs)

Donkie#235, Donkie#238, Donkie#240, Donkie#242, Donkie#246, Donkie#250, Donkie#251, Donkie#253, Donkie#254, Donkie#256, Donkie#263, Donkie#264, Donkie#265, Donkie#266, Donkie#267, Donkie#268, Donkie#269, Donkie#270, Donkie#272, Donkie#275, plus Donkie#257, Donkie#271, Donkie#274 (entries inserted by hand where patches collided textually with sibling PRs).

Structural PRs verified safe before merging:

Merged with fixes (8 PRs)

PR Fix applied
Donkie#202 Eryone part only — the eSun half was dropped per the upstream maintainer's own review (duplicate of already-merged data); added the cardboard spool type the title promised
Donkie#231 Title-cased 27 lowercase Spectrum color names; corrected 3 wrong hexes using official RAL codes (Navy Blue → RAL 5002 20214F, Lion Orange → RAL 2004 F44611, Royal Blue 8B008B4169E1)
Donkie#243 Restructured Deeplee entry to Matte {color_name} with finish: matte; dropped a French-language duplicate color
Donkie#247 Converted CRLF→LF; removed stray blank line
Donkie#248 Fixed silently-ignored spool_material key → spool_type; corrected brand spelling to SainSmart
Donkie#258 Rewrote schema-invalid "hex": "0F1217, #002789" as a hexes array with multi_color_direction: coaxial
Donkie#259 Applied the upstream reviewer's requested correction the author never pushed (Orange → "Sunny Orange" ED7334)
Donkie#241 Partial: only its six non-PLA Inland lines; PLA data superseded by Donkie#261's measured colors; dropped the mislabeled "PolyLite ASA" entry (a Polymaker product)

Skipped (4 PRs)

Validation

🤖 Generated with Claude Code

https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ


Generated by Claude Code

Ares9323 and others added 30 commits July 2, 2026 07:16
… FILAMENT (upstream PR Donkie#268)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
…board (upstream PR Donkie#202)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Etruscian and others added 6 commits July 2, 2026 07:22
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
…nkie#241, partial)

Only the non-PLA entries; PLA lines and the mislabeled PolyLite ASA entry
were dropped in favor of PR Donkie#261's measured data.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXFsv1PV9wVWGgwXGz2guJ
@sherrmann sherrmann merged commit 5518cf1 into main Jul 2, 2026
3 checks passed
@sherrmann sherrmann deleted the claude/upstream-pr-review-merge-kobghs branch July 2, 2026 08:16
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.