Skip to content

Commit

Permalink
Update WP_Async_Task dependency inclusion method.
Browse files Browse the repository at this point in the history
* Was using submodules, which were not included as part of the build step. Moving to composer
  • Loading branch information
jsainton-godaddy committed Feb 8, 2022
1 parent 6a21d01 commit 9cdab20
Show file tree
Hide file tree
Showing 12 changed files with 78 additions and 16 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/deploy-to-wp-org.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@master
- name: Build
run: |
composer install -o --no-dev --ignore-platform-reqs
- name: WordPress Plugin Deploy
uses: 10up/action-wordpress-plugin-deploy@stable
env:
Expand Down
3 changes: 0 additions & 3 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -1,3 +0,0 @@
[submodule "includes/vendor/wp-async-task"]
path = includes/vendor/wp-async-task
url = [email protected]:techcrunch/wp-async-task.git
3 changes: 3 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,9 @@ Below the text box is a button that will allow you to simply save all of that in

## Changelog ##

### 3.2.1 ###
* Resolve build process dependency error.

### 3.2.0 ###
* Add ability to edit Alt Text on media
* Update dependencies
Expand Down
5 changes: 3 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "gathercontent/wp-importer",
"description": "Imports items from GatherContent to your wordpress site",
"version": "3.2.0",
"version": "3.2.1",
"type": "wordpress-plugin",
"keywords": [],
"homepage": "http://www.gathercontent.com",
Expand All @@ -16,7 +16,8 @@
],
"minimum-stability": "dev",
"require": {
"php": ">=7.0"
"php": ">=7.0",
"techcrunch/wp-async-task": "dev-master"
},
"require-dev": {
"antecedent/patchwork": "2.1.*",
Expand Down
62 changes: 60 additions & 2 deletions composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions gathercontent-importer.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
* Plugin Name: GatherContent Plugin
* Plugin URI: http://www.gathercontent.com
* Description: Imports items from GatherContent to your wordpress site
* Version: 3.2.0
* Version: 3.2.1
* Author: GatherContent
* Requires PHP: 7.0
* Author URI: http://www.gathercontent.com
Expand Down Expand Up @@ -31,8 +31,8 @@
*/

// Useful global constants
define( 'GATHERCONTENT_VERSION', '3.2' );
define( 'GATHERCONTENT_ENQUEUE_VERSION', '3.2' );
define( 'GATHERCONTENT_VERSION', '3.2.1' );
define( 'GATHERCONTENT_ENQUEUE_VERSION', '3.2.1' );
define( 'GATHERCONTENT_SLUG', 'gathercontent-import' );
define( 'GATHERCONTENT_PLUGIN', __FILE__ );
define( 'GATHERCONTENT_URL', plugin_dir_url( __FILE__ ) );
Expand Down
2 changes: 0 additions & 2 deletions includes/classes/post-types/async-save-hook.php
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
<?php
namespace GatherContent\Importer\Post_Types;

require_once GATHERCONTENT_INC . 'vendor/wp-async-task/wp-async-task.php';

class Async_Save_Hook extends \WP_Async_Task {
protected $action = 'save_post';
protected $post_type = 'save_post';
Expand Down
2 changes: 0 additions & 2 deletions includes/classes/sync/async-base.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
use GatherContent\Importer\General;
use GatherContent\Importer\Debug;

require_once GATHERCONTENT_INC . 'vendor/wp-async-task/wp-async-task.php';

abstract class Async_Base extends \WP_Async_Task {

/**
Expand Down
2 changes: 2 additions & 0 deletions includes/functions/core.php
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,8 @@ function setup() {

spl_autoload_register( $n( 'autoload' ), false );

include_once GATHERCONTENT_PATH . 'vendor/autoload.php';

if ( is_admin() ) {
// We only need to do our work in the admin.
add_action( 'init', $n( 'init' ) );
Expand Down
1 change: 0 additions & 1 deletion includes/vendor/wp-async-task
Submodule wp-async-task deleted from 9bdbbf
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "gathercontent-importer",
"title": "GatherContent Plugin",
"description": "Imports items from GatherContent to your wordpress site",
"version": "3.1.13",
"version": "3.2.1",
"license": "GPLv2",
"homepage": "http://www.gathercontent.com",
"repository": {
Expand Down
3 changes: 3 additions & 0 deletions readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,9 @@ Below the text box is a button that will allow you to simply save all of that in

== Changelog ==

= 3.2.1 =
* Resolve build process dependency error.

= 3.2.0 =
* Add ability to edit Alt Text on media
* Update dependencies
Expand Down

0 comments on commit 9cdab20

Please sign in to comment.