Skip to content

Upgrade Version to 3.5#309

Merged
benbalter merged 68 commits intowp-document-revisions:mainfrom
NeilWJames:main
Feb 28, 2023
Merged

Upgrade Version to 3.5#309
benbalter merged 68 commits intowp-document-revisions:mainfrom
NeilWJames:main

Conversation

@NeilWJames
Copy link
Copy Markdown
Collaborator

@benbalter,
This pull request addresses #306, #307 and #308 as well as the fixes mentioned in the changes text.

As it is over a year since the plugin has been updated, it probaby needs to be released to WP. That code still contains #283.

Whilst I have had to drop tests for PHP 5.6 and 7.0 als the WP polyfill process fails for these versions (WPTnitTestCase::setUp needs to return void and this is not supported until PHP 7.1), I have added an explicit test set using PHP 7.1 and WP 5.1. These are the earliest versions that currently work with the test suite.

I have modified the minimum supported WP version from 4.6 to 4.9. I have done this for two reasons. Although it is not a tested version, there is a good chance that it works with that version. There are folks that are using 4.9 still to avoid Gutenburg still and I would try to help them, but I do not have a lot of interest to attempt support the earlier versions.

I have added some tests to cover both the original post_content format (simple integer) or 3.4 format (html comment) as well as increasing the coverage percentage.

Thanks,
Neil James

NeilWJames and others added 30 commits August 23, 2022 22:49
Updates the requirements on [dealerdirect/phpcodesniffer-composer-installer](https://github.com/PHPCSStandards/composer-installer) to permit the latest version.
- [Release notes](https://github.com/PHPCSStandards/composer-installer/releases)
- [Changelog](https://github.com/PHPCSStandards/composer-installer/blob/main/.github_changelog_generator)
- [Commits](PHPCSStandards/composer-installer@v0.7.0...v1.0.0)

---
updated-dependencies:
- dependency-name: dealerdirect/phpcodesniffer-composer-installer
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
…ct/phpcodesniffer-composer-installer-tw-1.0

Update dealerdirect/phpcodesniffer-composer-installer requirement from ^0.7 to ^1.0
Update dealerdirect/phpcodesniffer-composer-installer requirement
@NeilWJames NeilWJames requested a review from benbalter February 27, 2023 00:33
@codecov-commenter

This comment was marked as outdated.

@benbalter benbalter merged commit 5d9c764 into wp-document-revisions:main Feb 28, 2023
@benbalter
Copy link
Copy Markdown
Collaborator

Wow, thanks for this @NeilWJames. Updated the required tests. I'll push up a new tag, which should automatically deploy to wordpress.org. 🚀

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