-
Notifications
You must be signed in to change notification settings - Fork 21
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
Improve Version Handling in Build & Backend #1043
base: develop
Are you sure you want to change the base?
Conversation
1. use resource processing of gradle to put version numbers into version properties file in resources 2. use same approach to also include the version numbers in spring application banner 3. apply idea plugin in gradle to ensure the same output dirs are used in intellij 4. extract the calculation of scenarioo version numbers into own version.gradle file 5. use standard gradle version property in gradle files where appropriate
…ess' of https://github.com/scenarioo/scenarioo into feature/cleanup_gradle_config_version_mess � Conflicts: � scenarioo-server/build.gradle
|
||
ext.versionWithGitCommit = getVersionName(); | ||
apply plugin: 'idea' |
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.
why do we need this?
'aggregatedDataFormatVersion=' + aggregatedDataFormatVersion + '\n' + | ||
'documentationVersion=' + documentationVersion + '\n') | ||
/** | ||
* Replace properties in important resource files |
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.
I think java doc style should only be used on methods. I would use a /* single comment */ style here.
@@ -160,7 +152,7 @@ publishing { | |||
|
|||
groupId 'org.scenarioo' | |||
artifactId 'scenarioo-viewer' | |||
version versionWithGitCommit | |||
version scenariooViewerVersion |
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 is the same as archiveVersion, correct?
public class ApplicationVersionHolderTest { | ||
|
||
@Test | ||
public void initializeFromClassContext_loadsVersionProperlyFromVersionPropertiesFile() { |
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.
JUnit 5 does not require the public keyword
Improves the version handling in several aspects: