-
Notifications
You must be signed in to change notification settings - Fork 71
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
[INTERNAL] Adapt project configuration documentation #170
Conversation
Thodd
commented
Jul 24, 2019
- Cachebuster docu is moved to correct location
- *.properties file encoding option is documented
- Cachebuster docu is moved to correct location - *.properties file encoding option is documented
docs/pages/Configuration.md
Outdated
@@ -87,8 +87,9 @@ Some general information: | |||
+ `<virtual path>: <physical path>` (default `/: ./`): Any virtual path mapping can be defined here. | |||
*(Only available for projects of type `module`)* | |||
It is recommended that modules include their namespace in the virtual path and use the `/resources` prefix (e.g. `/resources/my/first/library/module-xy`). | |||
- `cachebuster`: | |||
- `signatureType`: `time` or `hash`. By default, the generated cachebuster info file signatures are based on timestamps (`time`). In setups like CI environments, a mechanism based on file hashes (`hash`) might be more reliable. | |||
- `propertiesFileEncoding`: |
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.
We should maybe also add the min. required CLI version for this (once released).
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.
Done. Expected to be released with UI5 CLI v1.7.0
.
docs/pages/Configuration.md
Outdated
@@ -87,8 +87,9 @@ Some general information: | |||
+ `<virtual path>: <physical path>` (default `/: ./`): Any virtual path mapping can be defined here. | |||
*(Only available for projects of type `module`)* | |||
It is recommended that modules include their namespace in the virtual path and use the `/resources` prefix (e.g. `/resources/my/first/library/module-xy`). | |||
- `cachebuster`: | |||
- `signatureType`: `time` or `hash`. By default, the generated cachebuster info file signatures are based on timestamps (`time`). In setups like CI environments, a mechanism based on file hashes (`hash`) might be more reliable. | |||
- `propertiesFileEncoding`: |
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.
- moved to "resources.configuration"
- rename according to impl.
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.
Done
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.
Do not merge, see comments by Matthias and Thorsten
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.
comments open
37f5210
to
816a8b3
Compare