-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(core): include serve target only for applications #30
Conversation
Do not include serve target for libraries as they cannot be run directly. Fixes #28
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, and definitely works. One small suggestion, albeit a bit of a nit pick.
Replace boolean `and` with ternary for clearer code.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
🎉 This PR is included in version 0.3.0-dev.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Do not include serve target for libraries as they cannot be run directly. Fixes #28 Co-authored-by: Ben Callaghan <bcallaghan@selectbankcard.com>
## [0.2.1-dev.1](v0.2.0...v0.2.1-dev.1) (2021-04-27) ### Bug Fixes * **core:** include serve target only for applications ([#30](#30)) ([d3a54ce](d3a54ce)), closes [#28](#28) * **core:** output paths still use windows separator on linux / mac ([ed1c53a](ed1c53a)) * **core:** use full project path in output directory ([7748f9c](7748f9c)), closes [#27](#27) ### Features * **repo:** enable semantic-release ([298c5af](298c5af))
## [0.2.1](v0.2.0...v0.2.1) (2021-04-27) ### Bug Fixes * **core:** include serve target only for applications ([#30](#30)) ([d3a54ce](d3a54ce)), closes [#28](#28) * **core:** output paths still use windows separator on linux / mac ([ed1c53a](ed1c53a)) * **core:** use full project path in output directory ([7748f9c](7748f9c)), closes [#27](#27) ### Features * **repo:** enable semantic-release ([298c5af](298c5af))
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Do not include serve target for libraries as they cannot be run directly.
Fixes #28