Skip to content

Internal: Update publish script [TMZ-803] #595

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ManorHazaz merged 2 commits into from
Jan 20, 2026
Merged

Internal: Update publish script [TMZ-803] #595

ManorHazaz merged 2 commits into from
Jan 20, 2026

Conversation

@hein-obox
Copy link
Member

@hein-obox hein-obox commented Dec 9, 2025 •
edited by gitstream-cm bot
Loading

✨ PR Description

Purpose: Simplify WordPress.org theme deployment pipeline by removing dry-run mode and consolidating SVN credential validation to improve reliability and reduce operational complexity.

Main changes:

  • Removed dry-run workflow input and separate dry-run script, consolidating to single deployment path with mandatory SVN credentials
  • Restructured publish script to copy files first, then delete removed files, and validate credentials before commit operations
  • Moved SVN authentication checks after directory preparation to fail earlier if secrets are misconfigured, preventing partial SVN state

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

@CLAassistant
Copy link

CLAassistant commented Dec 9, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@hein-obox hein-obox force-pushed the internal/TMZ-803-update-publish-script branch from 95a8cdc to 30d049c Compare December 9, 2025 12:58
@ManorHazaz ManorHazaz merged commit 69f81b9 into main Jan 20, 2026
42 checks passed
@ManorHazaz ManorHazaz deleted the internal/TMZ-803-update-publish-script branch January 20, 2026 10:44
github-actions bot pushed a commit that referenced this pull request Jan 20, 2026
@hein-obox @ManorHazaz @github-actions
Update publish theme to WordPress script (#595)
2a6d590 
Co-authored-by: Manor Hazaz <manorhazaz@gmail.com>
@github-actions github-actions bot mentioned this pull request Jan 20, 2026
Merged
ManorHazaz added a commit that referenced this pull request Jan 20, 2026
@github-actions @hein-obox @ManorHazaz
Update publish theme to WordPress script (#595) (#606)
d2f630e 
Co-authored-by: Hein van Vlastuin <94352322+hein-obox@users.noreply.github.com>
Co-authored-by: Manor Hazaz <manorhazaz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ManorHazaz ManorHazaz ManorHazaz approved these changes

Assignees

No one assigned

Labels

1 min review cp_3.4 🗑️ Deleted files

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hein-obox @CLAassistant @ManorHazaz