Skip to content

Commit d50a727

Browse files
committed
fix: missing logger init on generate command
1 parent 13ce931 commit d50a727

File tree

4 files changed

+11
-7
lines changed

4 files changed

+11
-7
lines changed

bin/commands/generate.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,8 @@ export default {
132132
* @returns {Promise<void>}
133133
*/
134134
async action(opts) {
135+
const logger = Logger.init();
136+
135137
const rules = getEnabledRules(opts.disableRule);
136138
const linter = opts.skipLint ? undefined : createLinter(rules);
137139

@@ -140,7 +142,7 @@ export default {
140142
linter?.report();
141143

142144
if (linter?.hasError()) {
143-
Logger.getInstance().error('Lint failed; aborting generation.');
145+
logger.error('Lint failed; aborting generation.');
144146
process.exit(1);
145147
}
146148

bin/commands/lint.mjs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,9 +79,9 @@ export default {
7979
* @returns {Promise<void>}
8080
*/
8181
async action(opts) {
82-
try {
83-
Logger.init(opts.transport);
82+
const logger = Logger.init(opts.transport);
8483

84+
try {
8585
const rules = getEnabledRules(opts.disableRule);
8686
const linter = createLinter(rules, opts.dryRun);
8787

@@ -91,7 +91,7 @@ export default {
9191

9292
process.exitCode = +linter.hasError();
9393
} catch (error) {
94-
Logger.getInstance().error('Error running the linter:', error);
94+
logger.error('Error running the linter:', error);
9595

9696
process.exitCode = 1;
9797
}

bin/commands/list.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ export const types = ['generators', 'rules', 'transports'];
1313
* @param {'generators' | 'rules' | 'transports'} type - The type of items to list.
1414
*/
1515
export default function list(type) {
16-
Logger.init();
16+
const logger = Logger.init();
1717

1818
const list =
1919
type === 'generators'
@@ -25,5 +25,5 @@ export default function list(type) {
2525
? availableRules
2626
: availableTransports;
2727

28-
Logger.getInstance().info(list);
28+
logger.info(list);
2929
}

src/logger/index.mjs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,10 @@ export const Logger = (() => {
1919
let instance;
2020

2121
/**
22+
* Inits the logger with the specified transport.
2223
*
2324
* @param {string} [transportName]
24-
* @returns {void}
25+
* @returns {LoggerInstance}
2526
*/
2627
function init(transportName = 'console') {
2728
const transport = transports[transportName];
@@ -31,6 +32,7 @@ export const Logger = (() => {
3132
}
3233

3334
instance = createLogger(transport);
35+
return instance;
3436
}
3537

3638
/**

0 commit comments

Comments
 (0)