Skip to content

updating composer, phpstan, phpcs#122

Merged
mambax7 merged 17 commits intoXOOPS:masterfrom
mambax7:feature/update_tools
Feb 7, 2026
Merged

updating composer, phpstan, phpcs#122
mambax7 merged 17 commits intoXOOPS:masterfrom
mambax7:feature/update_tools

Conversation

@mambax7
Copy link
Copy Markdown
Contributor

@mambax7 mambax7 commented Feb 6, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 6, 2026 22:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 updates the project’s PHP tooling configuration (Composer, PHPUnit, PHPStan, PHPCS) and refreshes CI configuration to test across multiple PHP versions.

Changes:

  • Updates composer.json dependency constraints and adds common developer scripts (lint/analyse/test/ci).
  • Modernizes PHPUnit configuration (testsuites + coverage/reporting settings).
  • Replaces the previous GitHub Actions workflow with a new matrix-based CI workflow, and expands Scrutinizer configuration.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
phpunit.xml.dist Updates PHPUnit settings, defines Unit/Integration suites, and switches to <coverage> reporting.
phpstan.neon Narrows analysis scope to src and updates exclusions/settings.
phpcs.xml.dist Switches to PSR-12 base ruleset and updates exclusions/sniff behavior.
composer.json Updates package metadata, version constraints, autoload/autoload-dev, and adds scripts.
.travis0.yml Adds a Travis configuration file (coverage upload + git-phpcs usage).
.scrutinizer.yml Expands Scrutinizer nodes to validate/update/check platform reqs across PHP versions.
.github/workflows/pr_tests.yml Removes the old CI workflow.
.github/workflows/ci.yml Adds a new CI workflow with dependency/coverage matrix and Codecov upload.

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

mambax7 and others added 6 commits February 6, 2026 17:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Refactor autoload configuration to separate autoload and autoload-dev sections.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 7 out of 8 changed files in this pull request and generated 2 comments.


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

mambax7 and others added 4 commits February 6, 2026 18:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 27 out of 28 changed files in this pull request and generated 2 comments.


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

@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 28 out of 29 changed files in this pull request and generated 2 comments.


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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mambax7 mambax7 merged commit 5383ce1 into XOOPS:master Feb 7, 2026
7 of 8 checks passed
mambax7 added a commit to mambax7/xmf that referenced this pull request Feb 8, 2026
- Pin all action references to full commit SHA hashes (supply-chain security)
- Fix PHPStan skip condition from lexicographic >= to explicit != exclusion
- Fix SonarCloud token check (secrets context, not env context)
- Add repository guard to Qodana workflow
- Fix CHANGELOG: move Ulid overhaul from v1.2.32 to v1.2.33-beta2 (correct release),
  add missing PRs XOOPS#118/XOOPS#119/XOOPS#122, correct 11 release dates to match GitHub releases
- Improve phpcs.xml: use severity=0 instead of exclude-pattern for line length

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants