Skip to content

feat: remove workshop.pl and consolidate on workshop.py#112

Merged
k-rister merged 1 commit into
masterfrom
remove-workshop-pl
May 5, 2026
Merged

feat: remove workshop.pl and consolidate on workshop.py#112
k-rister merged 1 commit into
masterfrom
remove-workshop-pl

Conversation

@k-rister
Copy link
Copy Markdown
Contributor

@k-rister k-rister commented May 4, 2026

Summary

  • Remove workshop.pl (2,208 lines of Perl)
  • Update CI workflows to use workshop.py exclusively
  • Update CLAUDE.md and README.md to remove Perl references

The Python version has been the default since rickshaw PR #805, and bin/_main always passes --workshop-script=workshop.py. Perl dependencies in workshop.json remain for the 4 legacy supported releases (2025.1-2025.4).

Test plan

  • Workshop CI passes with workshop.py invocations
  • Crucible-merged CI passes

🤖 Generated with Claude Code

Remove the legacy Perl implementation. The Python version has been the
default since rickshaw PR #805 and is what bin/_main always invokes.
Update CI workflows and documentation to reference workshop.py only.

Perl dependencies in workshop.json remain for the 4 legacy supported
releases (2025.1-2025.4) that still use workshop.pl at runtime.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@k-rister k-rister force-pushed the remove-workshop-pl branch from 8948ca3 to bcc098c Compare May 4, 2026 21:05
@k-rister k-rister merged commit d832c50 into master May 5, 2026
5838 of 5896 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Crucible Tracking May 5, 2026
@k-rister k-rister deleted the remove-workshop-pl branch May 5, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants