Fix runner use of moleculer.config function with default export #1284
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Description
When using moleculer-runner and a specified
moleculer.config.<ext>file it should support both of the following:The current flow of the runner is checking if the required
moleculer.config.<ext>is a function prior to the logic which determines if the configuration file has a default export. This prevents the function from being executed if it is a default export.This PR makes the following changes:
loadConfigFileeven if a configuration file was not foundloadConfigFiletomergeOptionsmergeOptionsinstead ofthis.configFilethis.configFileloadConfigFileso that the default export check precedes the function check and execution.mtssuffixes to the esm runnermoleculer.config.async.tsandmoleculer.config.tsfiles to the runner examples which validate typescript configurations and default exports for use with the runner💎 Type of change
🚦 How Has This Been Tested?
Tested using the newly created runner examples as well as the existing examples for both cjs and esm variants.
🏁 Checklist: