Skip to content

Release 2.0.0: PHP ^8.2, portphp ^2.0, GitHub Actions - #3

Merged
slashrsm merged 2 commits into
portphp:masterfrom
slashrsm:release/2.0.0
Jul 22, 2026
Merged

Release 2.0.0: PHP ^8.2, portphp ^2.0, GitHub Actions#3
slashrsm merged 2 commits into
portphp:masterfrom
slashrsm:release/2.0.0

Conversation

@slashrsm

Copy link
Copy Markdown
Contributor

Summary

Major modernization of the XML writer.

Breaking

  • php: ^8.2
  • portphp/portphp: ^2.0
  • Writer methods use : void return types

Other

  • GitHub Actions CI, Dependabot
  • README / docs URL updates; remove Travis/Scrutinizer
  • UPGRADE-2.0.md

Please do not merge until approved.

Breaking major: drop PHP < 8.2 and portphp 1.x.
Add void return types, Dependabot, modern README.

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR prepares the portphp/xml package for a 2.0.0 release by modernizing runtime requirements (PHP ^8.2, portphp/portphp ^2.0), updating the writer API to match the new core interfaces, and migrating CI from Travis/Scrutinizer to GitHub Actions.

Changes:

  • Bump platform/dependency requirements (PHP ^8.2, portphp/portphp ^2.0) and update docs/upgrade guide accordingly.
  • Update XmlWriter method signatures to use : void return types.
  • Replace legacy CI/QA tooling (Travis/Scrutinizer) with GitHub Actions and add Dependabot configuration.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
FileDescription
UPGRADE-2.0.mdAdds upgrade notes documenting breaking changes for 2.0.0.
src/XmlWriter.phpUpdates writer method return types to void to align with updated interfaces.
README.mdRefreshes package naming/badges and updates documentation URL and requirements summary.
phpspec.yml.distUpdates PhpSpec suite configuration for the new CI flow.
phpspec.yml.ciRemoves the CI-specific PhpSpec/coverage configuration.
composer.jsonUpdates PHP and dependency constraints; modernizes dev dependencies (PhpSpec ^8).
.travis.ymlRemoves Travis CI configuration.
.scrutinizer.ymlRemoves Scrutinizer configuration.
.github/workflows/checks.ymlAdds GitHub Actions workflow to run PhpSpec across supported PHP versions.
.github/dependabot.ymlAdds Dependabot updates for GitHub Actions and Composer dependencies.
.gitattributesUpdates export-ignore list to drop legacy CI files (but still contains a stale entry).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread.gitattributes
Comment on lines 5 to 7
CONTRIBUTING.md export-ignore
phpspec.yml.ci export-ignore
phpspec.yml.dist export-ignore

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.

@slashrsm
slashrsm requested a review from CopilotJuly 22, 2026 13:05

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@slashrsm
slashrsm requested a review from CopilotJuly 22, 2026 13:06

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@slashrsm
slashrsm merged commit b89551e into portphp:masterJul 22, 2026
4 of 5 checks passed
Sign up for freeto 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.

2 participants

@slashrsm