Changeset 5912

Timestamp:

08/15/2015 06:30:56 AM (10 years ago)

Author:

netweb

Message:

Tools: Remove banners added to minified CSS files and simplified JS banner

This changeset removes the banners that were previously added to minified CSS files as grunt-contrib-cssmin deprecated this in v0.11.0.

The banner added to minified JS files has also been simplified by removing the time/date so that unnecessarily merges/updates do not trigger unneeded changes when using version control. The banner added to minified JS files is now static without the date/time, this fixes an issue where the minified file would always change because of the compiled date/time even when there were no code changes, this was causing unneeded changes when merging/updating when using version control external to bbPress.

Props netweb. Fixes #2738

File:

• trunk/Gruntfile.js (modified) (3 diffs)

trunk/Gruntfile.js

@@ -116 +116 @@
                 expand: true,
                 ext: '.min.css',
-                src: BBP_LTR_CSS,
-                options: {
-                    banner: '/*! <%= pkg.name %> - v<%= pkg.version %> - ' +
-                    '<%= grunt.template.today("UTC:yyyy-mm-dd h:MM:ss TT Z") %> - ' +
-                    'https://wordpress.org/plugins/bbpress/ */'
-                }
+                src: BBP_LTR_CSS
             },
             rtl: {
@@ -128 +123 @@
                 expand: true,
                 ext: '.min.css',
-                src: BBP_RTL_CSS,
-                options: {
-                    banner: '/*! <%= pkg.name %> - v<%= pkg.version %> - ' +
-                    '<%= grunt.template.today("UTC:yyyy-mm-dd h:MM:ss TT Z") %> - ' +
-                    'https://wordpress.org/plugins/bbpress/ */'
-                }
+                src: BBP_RTL_CSS
             }
         },
@@ -285 +275 @@
             },
             options: {
-                banner: '/*! <%= pkg.name %> - v<%= pkg.version %> - ' +
-                '<%= grunt.template.today("UTC:yyyy-mm-dd h:MM:ss TT Z") %> - ' +
-                'https://wordpress.org/plugins/bbpress/ */\n'
+                banner: '/*! This file is automatically generated. */\n'
             }
         },