From 39eae5e8db89e4106a3541c2c2285d5b28772e3c Mon Sep 17 00:00:00 2001 From: Joe Hildebrand Date: Tue, 2 Jul 2024 09:01:10 -0600 Subject: [PATCH] Clean up rollup hacks. Fixes #531. --- CHANGELOG.md | 3 +++ bin/peggy-cli.js | 37 ------------------------------------- 2 files changed, 3 insertions(+), 37 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 13dd69b0..9b352d6c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,6 +14,9 @@ Released: TBD ### Bug fixes +- [#531](https://github.com/peggyjs/peggy/issues/531) Clean up rollup hacks + from CLI code. + ### Documentation 4.0.3 diff --git a/bin/peggy-cli.js b/bin/peggy-cli.js index cce9ab98..9f409910 100644 --- a/bin/peggy-cli.js +++ b/bin/peggy-cli.js @@ -736,43 +736,6 @@ class PeggyCLI extends Command { return this.run(); } } - - // For some reason, after running through rollup, typescript can't see - // methods from the base class. - - /** - * @param {string[]} [argv] - optional, defaults to process.argv - * @param {Object} [parseOptions] - optionally specify style of options with from: node/user/electron - * @param {string} [parseOptions.from] - where the args are from: 'node', 'user', 'electron' - * @return {PeggyCLI} `this` command for chaining - */ - parse(argv, parseOptions) { - return super.parse(argv, parseOptions); - } - - /** - * @param {Object} [configuration] - configuration options - * @return {PeggyCLI} `this` command for chaining, or stored configuration - */ - configureHelp(configuration) { - return super.configureHelp(configuration); - } - - /** - * @param {Object} [configuration] - configuration options - * @return {PeggyCLI} `this` command for chaining, or stored configuration - */ - configureOutput(configuration) { - return super.configureOutput(configuration); - } - - /** - * @param {Function} [fn] optional callback which will be passed a CommanderError, defaults to throwing - * @return {PeggyCLI} `this` command for chaining - */ - exitOverride(fn) { - return super.exitOverride(fn); - } } exports.PeggyCLI = PeggyCLI;