Skip to content

Conversation

@oschwald
Copy link
Member

@oschwald oschwald commented Dec 3, 2025

Summary

  • Add pre-flight checks to verify required tools and authentication before making any changes
  • Checks include: gh CLI authentication, repository access, git remote connectivity, and required commands (perl, php, phpize, pecl)

This ensures the release script fails early with clear error messages rather than failing partway through after making changes.

Test plan

  • Verify script syntax with bash -n dev-bin/release.sh
  • Review changes to ensure pre-flight checks are correct

🤖 Generated with Claude Code

@oschwald oschwald force-pushed the greg/eng-1733 branch 2 times, most recently from b1b05fe to 96ed252 Compare December 3, 2025 23:01
oschwald and others added 2 commits December 4, 2025 07:04
- Support pre-release versions in changelog parsing
- Quote date command substitution for defensive programming
- Fix shellcheck SC2162: add -r flag to read command
- Apply shfmt formatting
i386 support has been dropped upstream in setup-php due to deb.sury
removing i386 packages.

See: shivammathur/setup-php#1040 (comment)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@horgh horgh merged commit 1fe6b63 into main Dec 5, 2025
75 checks passed
@horgh horgh deleted the greg/eng-1733 branch December 5, 2025 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants