-
Notifications
You must be signed in to change notification settings - Fork 22
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] ui5 init: Improve error message #661
Conversation
lib/init/init.js
Outdated
@@ -49,6 +49,9 @@ function getProjectType(hasWebapp, hasSrc, hasTest) { | |||
let message = `Could not detect project type: ${errorReason}`; | |||
message += "Applications should only have a 'webapp' folder.\n"; | |||
message += "Libraries should only have a 'src' and (optional) 'test' folder."; |
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.
message += "Libraries should only have a 'src' and (optional) 'test' folder."; | |
message += "Libraries should only have an 'src' and (optional) 'test' folder."; |
lib/init/init.js
Outdated
@@ -49,6 +49,9 @@ function getProjectType(hasWebapp, hasSrc, hasTest) { | |||
let message = `Could not detect project type: ${errorReason}`; | |||
message += "Applications should only have a 'webapp' folder.\n"; | |||
message += "Libraries should only have a 'src' and (optional) 'test' folder."; | |||
message += "\n\n"; | |||
message += "In case you are about to start a new project, please refer to:\n"; |
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.
message += "In case you are about to start a new project, please refer to:\n"; | |
message += "If you are about to start a new project, please refer to:\n"; |
test/lib/init/init.js
Outdated
@@ -40,49 +40,50 @@ test("Init for library", async (t) => { | |||
}); | |||
}); | |||
|
|||
const GENERAL_ERROR_MESSAGE = `Applications should only have a 'webapp' folder. | |||
Libraries should only have a 'src' and (optional) 'test' folder. |
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.
Libraries should only have a 'src' and (optional) 'test' folder. | |
Libraries should only have an 'src' and (optional) 'test' folder. |
test/lib/init/init.js
Outdated
const GENERAL_ERROR_MESSAGE = `Applications should only have a 'webapp' folder. | ||
Libraries should only have a 'src' and (optional) 'test' folder. | ||
|
||
In case you are about to start a new project, please refer to: |
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.
In case you are about to start a new project, please refer to: | |
If you are about to start a new project, please refer to: |
Fixes: SAP/ui5-tooling#306