Skip to content

Release/4.4.8#150

Merged
sorinmarta merged 2 commits into
masterfrom
release/4.4.8
Jul 21, 2026
Merged

Release/4.4.8#150
sorinmarta merged 2 commits into
masterfrom
release/4.4.8

Conversation

@sorinmarta

Copy link
Copy Markdown
Contributor

No description provided.

@sorinmarta sorinmarta added run analysis Run the phpcs run tests Run the unit tests labels Jul 21, 2026
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The plugin release is updated from 4.4.7 to 4.4.8 across runtime metadata, README changelog content, spinner media queries, and PHPDoc annotations. No executable PHP logic or test behavior changes.

Changes

4.4.8 Release Updates

Layer / File(s) Summary
Version metadata
awpcp.php
The plugin header and runtime database version are updated to 4.4.8.
Release documentation and presentation
README.TXT, resources/css/*.css
The README adds the 4.4.8 changelog entry and removes the older 4.3.3 section; spinner media-query syntax is updated in both stylesheets.
4.4.8 PHPDoc annotations
frontend/*, includes/*, tests/suite/*
Placeholder @since annotations are replaced with 4.4.8 across implementation and test documentation.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided, so there is no meaningful summary to assess. Add a short description summarizing the release version bump and changelog updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and matches the release/version bump shown in the changeset.
Docstring Coverage ✅ Passed Docstring coverage is 84.62% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release/4.4.8

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 PHPStan (2.2.5)

PHPStan was skipped because the user-provided config is missing the required paths: directive.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.TXT`:
- Line 218: Update the changelog entry to hyphenate the compound modifier, using
“ad-approved user email” instead of “ad approved user email.”

In `@resources/css/awpcp-admin.css`:
- Line 1: Correct the malformed spinner media query in
resources/css/awpcp-admin.css at lines 1-1 and resources/css/awpcpstyle.css at
lines 1-1 by adding “and” between “print” and the min-device-pixel-ratio
condition, while preserving the existing 2x spinner background rule.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5cd3e6-7b8d-4b0c-b935-8c87cb8297e3

📥 Commits

Reviewing files that changed from the base of the PR and between a74ede8 and d97c61b.

⛔ Files ignored due to path filters (1)
  • .github/workflows/release.yml is excluded by !**/*.yml
📒 Files selected for processing (13)
  • README.TXT
  • awpcp.php
  • frontend/class-query.php
  • frontend/shortcode.php
  • includes/class-authentication-redirection-handler.php
  • includes/listings/class-delete-listing-event-listener.php
  • includes/models/payment-transaction.php
  • includes/regions-api.php
  • resources/css/awpcp-admin.css
  • resources/css/awpcpstyle.css
  • tests/suite/Payments/Gateways/test-PayPalStandard.php
  • tests/suite/Payments/test-payment-transaction.php
  • tests/suite/functions/test-email-functions.php

Comment thread README.TXT
Comment thread resources/css/awpcp-admin.css
@sorinmarta
sorinmarta merged commit 4088336 into master Jul 21, 2026
13 checks passed
@sorinmarta
sorinmarta deleted the release/4.4.8 branch July 21, 2026 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run analysis Run the phpcs run tests Run the unit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant