Skip to content

Conversation

@dpanta94
Copy link
Member

@dpanta94 dpanta94 commented Dec 16, 2025

Summary

Introduces a run method. Enabling shepherd to execute a task synchronously using Action Scheduler.
It basically re-uses the same method that AS calls itself when you click on an action Run from the admin list table.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • ⚡ Performance improvement
  • ♻️ Code refactoring
  • 🧪 Test improvements
  • 🔧 Chore (build process, dependencies, etc.)

Related Issues

Closes #

Pre-Submission Checklist

Code Quality

  • PHPStan analysis passes: composer test:analysis
  • PHP compatibility check passes: composer compatibility
  • Coding standards pass: vendor/bin/phpcs
  • All tests pass: slic run wpunit && slic run integration
  • No debug code (var_dump, error_log, etc.) left in production code
  • No commented-out code unless specifically needed for reference
  • Documentation updated for any new features or changed behavior
  • New tests have been added for new functionality
  • All existing tests continue to pass

📖 Read the full contributing guidelines: CONTRIBUTING.md

@dpanta94 dpanta94 self-assigned this Dec 16, 2025
@dpanta94 dpanta94 added the enhancement New feature or request label Dec 16, 2025
@dpanta94 dpanta94 requested a review from lucatume December 16, 2025 17:57
@dpanta94 dpanta94 requested a review from estevao90 December 16, 2025 17:58
Copy link
Contributor

@lucatume lucatume left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor stuff, good work here.

Copy link

@estevao90 estevao90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

@dpanta94 dpanta94 requested a review from lucatume December 17, 2025 15:19
@dpanta94 dpanta94 merged commit aadb440 into main Dec 17, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants