Backport changes from master to v2 branch #130
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fix: correct PHP notice on register-rest-route in WordPress v5.5.0
Update PR template
Updated WP tested versions
Display skipped test as success
Fixed new session on every request.
Also use the WP_ENV constant
Add settings link to plugins page
Release 2.6.4
- Switch from standard PHPUnit to yoast polyfill. - Setup sniff dependencies
Version 2 requires PHP7.4
`void` can't be used as a return type in PHP 7.0. The Yoast runner has helpers for this, so switching to those. See: https://github.com/Yoast/PHPUnit-Polyfills#testcases
v4 of the library introduced a few breaking changes. 1) GenerateErrorWrappr requries line to be an int 2) Rollbar\RollbarLogger:: log is now a wrapper around `report` that doesn't return the value that log used to return
Focus on the current stable WordPress, but still include upcoming WP
Add script for updating dependecies
Fix unit tests and PHP8.1+ compatability
Bumps [semver](https://github.com/npm/node-semver) from 5.7.1 to 5.7.2. - [Release notes](https://github.com/npm/node-semver/releases) - [Changelog](https://github.com/npm/node-semver/blob/v5.7.2/CHANGELOG.md) - [Commits](npm/node-semver@v5.7.1...v5.7.2) --- updated-dependencies: - dependency-name: semver dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
….7.2 Bump semver from 5.7.1 to 5.7.2
update to 2.7.0
Rollbar/WordPress/UI creates checkboxes for booleans and stores the state as "1" and "0". The PHP8 version of https://github.com/rollbar/rollbar-php expects actual booleans. this updates the config builder to convert "1" and "0" into TRUE and FALSE. Fixes: #119
Cast "1" and "0" as booleans when they are expected as booleans.
Update to version 2.7.1
Bumps [cross-spawn](https://github.com/moxystudio/node-cross-spawn) from 6.0.5 to 6.0.6. - [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/v6.0.6/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v6.0.5...v6.0.6) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
…awn-6.0.6 Bump cross-spawn from 6.0.5 to 6.0.6
|
It will probably take some work to get the tests running. I don't want to create a complex web of PRs so I would like to resolve that in the v2 branch after this is merged. |
brianr
approved these changes
Aug 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the change
This PR takes the work that has happened in the master branch since the v2 branch was first created and moves it to the v2 branch so it is retained.
Type of change
Related issues
None
Checklists
Development
Code review