Changeset 57758

Timestamp:

03/04/2024 11:23:59 AM (5 months ago)

Author:

desrosj

Message:

Build/Test Tools: Fix the precommit:emoji script.

GitHub recently sunset support for Subversion, causing the precommit:emoji Grunt script to break. Since there’s no direct replacement for svn ls in Git, this has been replaced with a query through the GitHub CLI.

This also adds a step in the workflow that tests the build process to run the precommit:emoji script to ensure no changes to built files are missed when updating the Twemoji library in the future.

Follow up to [57626].

Props kraftbj, peterwilsoncc, swissspidy.
Fixes #60520. See #57600.

Location:

trunk

Files:

• .github/workflows/callable-test-core-build-process.yml (modified) (1 diff)
• Gruntfile.js (modified) (1 diff)

trunk/.github/workflows/callable-test-core-build-process.yml

@@ -63 +63 @@
         run: npm ci
 
+
+
+
+
+
       - name: Build WordPress to run from ${{ inputs.directory }}
         run: npm run build${{ inputs.directory == 'src' && ':dev' || '' }}

trunk/Gruntfile.js

@@ -1048 +1048 @@
                             match: /\/\/ START: emoji arrays[\S\s]*\/\/ END: emoji arrays/g,
                             replacement: function() {
-                                var regex, files,
+                                var regex, files,
                                     partials, partialsSet,
-                                    entities, emojiArray;
+                                    entities, emojiArray,
+                                    apiResponse, query;
 
                                 grunt.log.writeln( 'Fetching list of Twemoji files...' );
 
+
+
+
+
+
+
                                 // Fetch a list of the files that Twemoji supplies.
-                                files = spawn( 'svn', [ 'ls', 'https://github.com/twitter/twemoji.git/trunk/assets/svg' ] );
+                                query = 'query={repository(owner: "jdecked", name: "twemoji") {object(expression: "v15.0.3:assets/svg") {... on Tree {entries {name}}}}}';
+                                files = spawn( 'gh', [ 'api', 'graphql', '-f', query] );
+
                                 if ( 0 !== files.status ) {
                                     grunt.fatal( 'Unable to fetch Twemoji file list' );
                                 }
 
-                                entities = files.stdout.toString();
+                                try {
+                                    apiResponse = JSON.parse( files.stdout.toString() );
+                                } catch ( e ) {
+                                    grunt.fatal( 'Unable to parse Twemoji file list' );
+                                }
+                                entities = apiResponse.data.repository.object.entries;
+                                entities = entities.reduce( function( accumulator, val ) { return accumulator + val.name + '\n'; }, '' );
 
                                 // Tidy up the file list.