Skip to content

Commit

Permalink
feat: kinda working with simplified config
Browse files Browse the repository at this point in the history
  • Loading branch information
laras126 committed Oct 15, 2020
1 parent be9f07b commit 80fa8d1
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 24 deletions.
22 changes: 2 additions & 20 deletions css-audit.config.js
Original file line number Diff line number Diff line change
@@ -1,28 +1,10 @@
module.exports = {
"format": "html",
"filename": "wp-admin",
"format": "cli-table",
"audits": [
"colors",
"important",
"display-none",
"selectors",
"media-queries",
[
"property-values",
{
"options": [ "font-size" ]
}
],
[
"property-values",
{
"options": [
"padding-top",
"padding-bottom",
"padding-left",
"padding-right"
]
}
]
"media-queries"
]
};
23 changes: 19 additions & 4 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ const path = require( 'path' );
*/
const { formatReport } = require( './utils/format-report' );
const { getArgFromCLI, getFileArgsFromCLI, getHelp } = require( './utils/cli' );

const input = getFileArgsFromCLI();

if ( getArgFromCLI( '--help' ) || ! input.length ) {
console.log( getHelp() ); // eslint-disable-line no-console
process.exit( 0 );
Expand All @@ -32,11 +32,23 @@ input.forEach( ( file ) => {
} );
} );

const audits = [];

const config = require( '../css-audit.config.js' );
const usingConfig = getArgFromCLI( '--config' );

if ( usingConfig ) {

// TODO: Support value for config arg, and default to css-audit.config filename
config.audits.forEach( audit => {
// TODO: check for and support property-values array
audits.push( require( `./audits/${audit}` )( cssFiles ) );
});
}

const runAll = getArgFromCLI( '--all' );
const runRecommended = getArgFromCLI( '--recommended' );

const audits = [];

if ( runAll || runRecommended || getArgFromCLI( '--colors' ) ) {
audits.push( require( './audits/colors' )( cssFiles ) );
}
Expand All @@ -63,4 +75,7 @@ if ( !! getArgFromCLI( '--property-values' ) ) {

const reports = audits.flat().filter( Boolean );

console.log( formatReport( reports, getArgFromCLI( '--format' ) ) ); // eslint-disable-line no-console
// TODO: This needs more thought
const format = usingConfig ? config.format : getArgFromCLI( '--format' );

console.log( formatReport( reports, format ) ); // eslint-disable-line no-console

0 comments on commit 80fa8d1

Please sign in to comment.