Merge pull request #1086 from cloudinary/github-actions

Replace Travis CI with GitHub actions

Author: aatanasov-cloudinary

.eslintrc.json

@@ -24,6 +24,7 @@
     "no-nested-ternary": "off",
     "jsx-a11y/click-events-have-key-events": "off",
     "react-hooks/rules-of-hooks": "error",
-    "react-hooks/exhaustive-deps": "warn"
+    "react-hooks/exhaustive-deps": "warn",
+    "@wordpress/no-global-event-listener": "off"
   }
 }

.github/workflows/ci.yml

@@ -0,0 +1,43 @@
+name: CI
+
+on:
+  push:
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        php-version: [ '5.6', '7.4', '8.3' ]
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php-version }}
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '16'
+          cache: 'npm'
+
+      - name: Cache Composer
+        uses: actions/cache@v4
+        with:
+          path: ~/.composer/cache
+          key: ${{ runner.os }}-composer-${{ matrix.php }}-${{ hashFiles('**/composer.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-composer-${{ matrix.php }}-
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run linting
+        run: npm run lint
+
+      - name: Run build
+        run: npm run build

.travis.yml

@@ -39,7 +39,7 @@
 if ( version_compare( phpversion(), '5.6', '>=' ) ) {
 	require_once __DIR__ . '/instance.php';
 	register_activation_hook( __FILE__, array( 'Cloudinary\Utils', 'install' ) );
-} else {
+} else { // phpcs:ignore Universal.ControlStructures.DisallowLonelyIf.Found
 	if ( defined( 'WP_CLI' ) ) {
 		WP_CLI::warning( php_version_text() );
 	} else {

cloudinary.php

@@ -287,7 +287,6 @@ public function init() {
 			$this->registered_cache_points[ $post->post_title ] = $post;
 		}
 		do_action( 'cloudinary_cache_init_cache_points' );
-
 	}
 
 	/**
@@ -526,7 +525,7 @@ public function get_cache_items( $cache_point_id_url, $id_only = false ) {
 			do {
 				$found = $posts->get_posts();
 				$items = array_merge( $items, $found );
-				$params['paged'] ++;
+				++$params['paged'];
 				$posts = new \WP_Query( $params );
 			} while ( $posts->have_posts() );
 		}
@@ -884,7 +883,7 @@ public function query_cached_items( $urls ) {
 					$found_posts[ $url ] = $meta[ self::META_KEYS['cached_urls'] ][ $url ];
 				}
 			}
-			$params['paged'] ++;
+			++$params['paged'];
 			$posts = new \WP_Query( $params );
 		} while ( $posts->have_posts() );
 

php/cache/class-cache-point.php

@@ -203,7 +203,7 @@ public function register_admin( $page ) {
 		$page_handle = add_menu_page(
 			$page['page_title'],
 			$page['menu_title'],
-			$page['capability'],
+			$page['capability'], // phpcs:ignore WordPress.WP.Capabilities.Undetermined
 			$page['slug'],
 			'',
 			$page['icon'],
@@ -245,7 +245,7 @@ public function register_admin( $page ) {
 				$page['slug'],
 				$page_title,
 				$menu_title,
-				$capability,
+				$capability, // phpcs:ignore WordPress.WP.Capabilities.Undetermined
 				$render_slug,
 				$render_function,
 				$position

php/class-admin.php

@@ -1365,7 +1365,7 @@ public function is_asset( $url ) {
 		global $wpdb;
 
 		// Bail early if it's not an URL.
-		if ( empty( parse_url( $url, PHP_URL_HOST ) ) ) {
+		if ( empty( parse_url( $url, PHP_URL_HOST ) ) ) { // phpcs:ignore WordPress.WP.AlternativeFunctions.parse_url_parse_url
 			return false;
 		}
 

php/class-assets.php

@@ -615,7 +615,6 @@ protected function get_plugins_table() {
 			'title'      => __( 'Plugin', 'cloudinary' ),
 			'root_paths' => $rows,
 		);
-
 	}
 
 	/**
@@ -987,7 +986,7 @@ public function add_content_cache_paths() {
 		if ( ! is_admin() ) {
 			// Exclude content replacement in admin.
 			$this->add_cache_paths( 'cache_content', 'content_files', 'cache_all_content' );
-		};
+		}
 	}
 
 	/**

php/class-cache.php

@@ -157,7 +157,6 @@ public function register_assets() {
 	 * Enqueue Assets.
 	 */
 	public function enqueue_assets() {
-
 	}
 
 	/**

php/class-delivery-feature.php

@@ -1964,7 +1964,7 @@ protected function filter_unknown_urls( $urls ) {
 			return $urls;
 		}
 
-		$known_lookup = array_flip( $known_keys );
+		$known_lookup      = array_flip( $known_keys );
 		$potential_unknown = array_diff( $urls, $known_keys );
 
 		if ( empty( $potential_unknown ) ) {