chore: fix prettier format for built files (#4258)

* chore: fix prettier format for built files * catch
dequelabs · Nov 28, 2023 · f1af598 · f1af598
1 parent f7b8d5e
commit f1af598
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 15 deletions.
diff --git a/build/tasks/aria-supported.js b/build/tasks/aria-supported.js
@@ -15,6 +15,7 @@ module.exports = function (grunt) {
        * as `axe` does not exist until grunt task `build:uglify` is complete,
        * hence cannot be required at the top of the file.
        */
+      const done = this.async();
       const { langs } = this.options();
       const fileNameSuffix = langs && langs.length > 0 ? `${langs[0]}` : '';
       const axe = require(`../../axe${fileNameSuffix}`);
@@ -69,10 +70,16 @@ module.exports = function (grunt) {
       const destFile = this.data.destFile;
       // Format the content so Prettier doesn't create a diff after running.
       // See https://github.com/dequelabs/axe-core/issues/1310.
-      const formattedContent = format(content, destFile);
-
-      // write `aria supported` file contents
-      grunt.file.write(destFile, formattedContent);
+      format(content, destFile)
+        .then(formattedContent => {
+          // write `aria supported` file contents
+          grunt.file.write(destFile, formattedContent);
+          done();
+        })
+        .catch(err => {
+          console.error(err.message);
+          done(false);
+        });
 
       /**
        * Get list of aria attributes, from `aria-query`

diff --git a/build/tasks/configure.js b/build/tasks/configure.js
@@ -1,15 +1,15 @@
 /*eslint-env node */
 'use strict';
-var buildRules = require('../configure');
-var format = require('../shared/format');
+const buildRules = require('../configure');
+const format = require('../shared/format');
 
 module.exports = function (grunt) {
   grunt.registerMultiTask(
     'configure',
     'Task for configuring rules and checks',
     function () {
-      var done = this.async();
-      var options = this.options({
+      const done = this.async();
+      const options = this.options({
         rules: ['lib/rules/**/*.json'],
         checks: ['lib/checks/**/*.json'],
         misc: ['lib/misc/**/*.json'],
@@ -18,7 +18,7 @@ module.exports = function (grunt) {
       });
 
       this.files.forEach(function (file) {
-        var match = file.dest.auto.match(/\.([a-z]{2,3})\.js/);
+        const match = file.dest.auto.match(/\.([a-z]{2,3})\.js/);
         if (match) {
           options.locale = match[1];
         }
@@ -28,12 +28,15 @@ module.exports = function (grunt) {
 
           // Format the content so Prettier doesn't create a diff after running.
           // See https://github.com/dequelabs/axe-core/issues/1310.
-          const descriptionsContent = format(
-            result.descriptions,
-            file.dest.descriptions
-          );
-          grunt.file.write(file.dest.descriptions, descriptionsContent);
-          done();
+          format(result.descriptions, file.dest.descriptions)
+            .then(descriptionsContent => {
+              grunt.file.write(file.dest.descriptions, descriptionsContent);
+              done();
+            })
+            .catch(err => {
+              console.error(err.message);
+              done(false);
+            });
         });
       });
     }