Skip to content

Fix VIP compatibility#987

Merged
dkotter merged 3 commits intodevelopfrom
fix/791-make-plugin-vip-compliant
Aug 26, 2025
Merged

Fix VIP compatibility#987
dkotter merged 3 commits intodevelopfrom
fix/791-make-plugin-vip-compliant

Conversation

@av3nger
Copy link
Contributor

@av3nger av3nger commented Aug 25, 2025

Description of the Change

Removes the RSS Importer WP-CLI command, as this command was there for legacy reasons and is no longer used.

Closes #973

How to test the Change

  • Verify file changes look accurate

Changelog Entry

Developer - Remove the RSS Importer WP-CLI command.

Credits

Props: @av3nger

Checklist:

@av3nger av3nger requested review from a team, dkotter and jeffpaul as code owners August 25, 2025 00:26
@av3nger av3nger self-assigned this Aug 25, 2025
@github-actions github-actions bot added the needs:feedback This requires reporter feedback to better understand the request. label Aug 25, 2025
@github-actions
Copy link

@av3nger thanks for the PR! Could you please fill out the PR template with description, changelog, and credits information so that we can properly review and merge this?

@av3nger av3nger added needs:code-review This requires code review. and removed needs:feedback This requires reporter feedback to better understand the request. labels Aug 25, 2025
@dkotter
Copy link
Collaborator

dkotter commented Aug 25, 2025

So this file was purposely not updated coming out of #972 because it is a legacy thing that is better off just being removed. A separate issue was opened for this (#973). I'd suggest updating this PR to remove this file instead of updating the use of wp_remote_get

@av3nger
Copy link
Contributor Author

av3nger commented Aug 26, 2025

@dkotter , thank you for the insight. I've double checked and the class is not used anywhere in the plugin. I have removed it from the plugin.

@dkotter dkotter linked an issue Aug 26, 2025 that may be closed by this pull request
1 task
@github-actions github-actions bot added this to the 3.7.0 milestone Aug 26, 2025
@dkotter dkotter merged commit 3d147dd into develop Aug 26, 2025
19 of 20 checks passed
@dkotter dkotter deleted the fix/791-make-plugin-vip-compliant branch August 26, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove legacy RSS Importer WP-CLI command

2 participants