-
Notifications
You must be signed in to change notification settings - Fork 19
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
Deterministic Code Generation #65
Changes from all commits
828d4db
e45dc84
6d933c7
cad14cb
65a6acb
9d7751b
0dc7777
032ce19
e39ae06
bcb69eb
a8e0849
54e8a1c
19796be
20040e1
f5f3734
67a1aef
e3642cc
341ef8f
02f4fa5
46d8536
670949d
6018c53
6a6443e
01a280d
af94ada
5b073e1
980f569
85d89bf
87c8e53
8b9afb3
b724e10
48452dc
064c5ea
847e956
66d37c0
72e6bf0
b406b37
5a4f38f
e10becd
fa34361
fef7b18
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
# | ||
# Copyright 2017 EntIT Software LLC, a Micro Focus company, L.P. | ||
# Licensed under the Apache License, Version 2.0 (the "License"); | ||
# you may not use this file except in compliance with the License. | ||
# You may obtain a copy of the License at | ||
# http://www.apache.org/licenses/LICENSE-2.0 | ||
# | ||
# Unless required by applicable law or agreed to in writing, software | ||
# distributed under the License is distributed on an "AS IS" BASIS, | ||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
# See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
# | ||
|
||
stages: | ||
- publish | ||
|
||
publish-sdk: | ||
image: maven:3-jdk-8-slim | ||
stage: publish | ||
only: | ||
- tags | ||
script: | ||
- mvn versions:set -DnewVersion=${CI_COMMIT_TAG} | ||
- mvn clean install -DskipTests | ||
- mvn deploy --settings .m2/settings.xml -DskipTests -Dmaven.install.skip=true | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
<!-- | ||
|
||
Copyright 2017 EntIT Software LLC, a Micro Focus company, L.P. | ||
Licensed under the Apache License, Version 2.0 (the "License"); | ||
you may not use this file except in compliance with the License. | ||
You may obtain a copy of the License at | ||
http://www.apache.org/licenses/LICENSE-2.0 | ||
|
||
Unless required by applicable law or agreed to in writing, software | ||
distributed under the License is distributed on an "AS IS" BASIS, | ||
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
See the License for the specific language governing permissions and | ||
limitations under the License. | ||
|
||
--> | ||
<settings xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.1.0 http://maven.apache.org/xsd/settings-1.1.0.xsd" | ||
xmlns="http://maven.apache.org/SETTINGS/1.1.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"> | ||
<servers> | ||
<server> | ||
<id>snapshot-repository</id> | ||
<username>${env.MAVEN_REPO_USER}</username> | ||
<password>${env.MAVEN_REPO_PASS}</password> | ||
</server> | ||
</servers> | ||
</settings> |
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
# Organize Import Order | ||
5=liquibase | ||
4=lombok | ||
3=com | ||
2=org | ||
1=javax | ||
0=java |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -75,6 +75,7 @@ | |
|
||
<commons-configuration.version>2.0</commons-configuration.version> | ||
<commons-beanutils.version>1.9.2</commons-beanutils.version> | ||
<commons-lang.version.version>3.8.1</commons-lang.version.version> | ||
|
||
<maven-release.version>2.5.3</maven-release.version> | ||
<maven-surefire.version>2.19.1</maven-surefire.version> | ||
|
@@ -188,6 +189,27 @@ | |
<!-- mvn com.mycila:license-maven-plugin:format (add headers if missing) --> | ||
<!-- mvn com.mycila:license-maven-plugin:remove (remove existing header) --> | ||
<plugins> | ||
<plugin> | ||
<groupId>com.diffplug.spotless</groupId> | ||
<artifactId>spotless-maven-plugin</artifactId> | ||
<version>1.17.0</version> | ||
<configuration> | ||
<java> | ||
<trimTrailingWhitespace/> | ||
<removeUnusedImports/> | ||
<includes> | ||
<include>sdk-generate-entity-models-maven-plugin/src/**/java/**/*.java</include> | ||
</includes> | ||
<eclipse> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I like the idea of using a code formatter, however will this work (integrate) with IntelliJ, we use that internally? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we also use intellij at emundo 😉 and all our projects has a build step in our ci to enforce correct formatting. |
||
<file>${session.executionRootDirectory}/eclipse-formatter.xml</file> | ||
<version>4.7.1</version> | ||
</eclipse> | ||
<importOrder> | ||
<file>${session.executionRootDirectory}/importorder.properties</file> | ||
</importOrder> | ||
</java> | ||
</configuration> | ||
</plugin> | ||
<plugin> | ||
<groupId>com.mycila</groupId> | ||
<artifactId>license-maven-plugin</artifactId> | ||
|
@@ -351,12 +373,12 @@ | |
<distributionManagement> | ||
<repository> | ||
<id>release-repository</id> | ||
<url>https://oss.sonatype.org/service/local/staging/deploy/maven2</url> | ||
<url>${env.MAVEN_RELEASE_REPOSITORY}</url> | ||
</repository> | ||
<snapshotRepository> | ||
<id>snapshot-repository</id> | ||
<name>Maven2 Snapshot Repository</name> | ||
<url>https://oss.sonatype.org/content/repositories/snapshots</url> | ||
<url>${env.MAVEN_SNAPSHOT_REPOSITORY}</url> | ||
<uniqueVersion>false</uniqueVersion> | ||
</snapshotRepository> | ||
</distributionManagement> | ||
|
Large diffs are not rendered by default.
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.
these for comment lines need to be next to the relevant maven plugin