This repository has been archived by the owner on Dec 24, 2024. It is now read-only.
fix(node): use provider creators #67
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Instead of using a function to generate the provider, I changed it for a provider creator, with a few extra options. BREAKING CHANGE: The `appConfiguration` function is now a provider creator and its signature changed
Instead of having `errorHandler` and `errorHandlerWithOptions`, they have been merged into one single provider creator. BREAKING CHANGE: The provider `errorHandlerWithOptions` doesn't exist anymore.
Instead of having `pathUtils` and `pathUtilsWithHome`, they have been merged into one single provider creator. BREAKING CHANGE: The provider `pathUtilsWithHome` doesn't exist anymore.
Instead of having `logger`, `loggerWithOptions`, `appLogger` and `appLoggerWithOptions`, the `WithOptions` providers were merged into the other ones using provider creators. BREAKING CHANGE: The providers `loggerWithOptions` and `appLoggerWithOptions` were removed.
Pull Request Test Coverage Report for Build 173449908
💛 - Coveralls |
Merged
github-actions bot
pushed a commit
that referenced
this pull request
Jul 19, 2020
# [4.0.0](3.0.4...4.0.0) (2020-07-19) ### Bug Fixes * add a package.json with the type 'module' on the ESM directory ([9e1521b](9e1521b)) * **node/logger:** use provider creators ([c2197a3](c2197a3)) * make all class properties private/protected with public getters ([f78a7e6](f78a7e6)), closes [#68](#68) * **node/errorhandler:** use a provider creator ([b53b71e](b53b71e)) * migrate from ESDoc to JSDoc ([65ab6cd](65ab6cd)) * remove 'main' from package.json ([515165a](515165a)) * use 'named' exports on the folders' indexes ([1ce2f2c](1ce2f2c)) * **node:** remove the providers that no longer exist ([544c9e8](544c9e8)) * **node:** use provider creators ([7f54d4e](7f54d4e)), closes [#67](#67) * **node/appconfiguration:** make all the class properties private/protected ([eab4cab](eab4cab)) * **node/appconfiguration:** use a provider creator ([ece3f9d](ece3f9d)) * **node/errorhandler:** make the properties private/protected ([82c7aa2](82c7aa2)) * **node/pathutils:** use a provider creator ([100fb44](100fb44)) * **shared/apiclient:** make the properties private/protected ([da216d3](da216d3)) * **shared/deepAssign:** use spread instead of .freeze ([bb8503c](bb8503c)) ### Features * add ESM version ([aebdab6](aebdab6)), closes [#61](#61) * add the deepAssign module ([d766a49](d766a49)), closes [#63](#63) * add the deepAssign module ([1e9247b](1e9247b)) * add the new Jimple wrappers ([67a8e6e](67a8e6e)), closes [#66](#66) * add the new Jimple wrappers ([0afc6a3](0afc6a3)) * add warn alias for warning on the Logger ([38cf90c](38cf90c)) * fix the the types for the APIClient and validate with ts-check ([346449b](346449b)) * **node/environmentutils:** use a provider creator ([c6821f9](c6821f9)) * **node/packageInfo:** use a provider creator ([fe66556](fe66556)) * **node/rootRequire:** use a provider creator ([9da0d2f](9da0d2f)) ### BREAKING CHANGES * **shared/apiclient:** The method `flattenEndpoints` was removed; it was just a wrapper for a call to `ObjectUtils`. * **node/errorhandler:** The properties for `.appLogger` and `.eventNames` are not longer accesible. * **node/appconfiguration:** The properties for `.environmentUtils` and `rootRequire` are not longer accessible. * **node/logger:** The providers `loggerWithOptions` and `appLoggerWithOptions` were removed. * **node/pathutils:** The provider `pathUtilsWithHome` doesn't exist anymore. * **node/errorhandler:** The provider `errorHandlerWithOptions` doesn't exist anymore. * **node/appconfiguration:** The `appConfiguration` function is now a provider creator and its signature changed * The error method now adds the status to the returned error message
|
🎉 This PR is included in version 4.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
What does this PR do?
This is a follow up for #66 :
providerCreators.This is super breaking as all the "providers generators" were removed.
How should it be tested manually?