Skip to content
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 maven build stability and unify pom format #151

Merged
merged 2 commits into from
Jan 4, 2021
Merged

improve maven build stability and unify pom format #151

merged 2 commits into from
Jan 4, 2021

Conversation

oldratlee
Copy link
Member

@oldratlee oldratlee commented Jan 3, 2021

  • improve maven build stability
    • fix maven-resources-plugin encoding warning
      [WARNING] Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
    • unify maven-compiler-plugin config: via properties(project.build.sourceEncoding/maven.compiler.source)
    • upgrade/unify maven-resources-plugin/maven-compiler-plugin version
  • unify pom format
    • unify modelVersion element to order first
    • add a blank line between artifactId element and parent/modelVersion
    • add a blank line before dependencies element

- fix maven-resources-plugin encoding warning
  [WARNING] Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
- unify maven-compiler-plugin config: via properties(project.build.sourceEncoding/maven.compiler.source)
- upgrade/unify maven-resources-plugin/maven-compiler-plugin version
- unify modelVersion element to order first
- add a blank line between artifactId element and parent/modelVersion
- add a blank line before dependencies element
@oldratlee oldratlee changed the title improve maven build stability improve maven build stability and unify pom format Jan 3, 2021
@significantfrank significantfrank merged commit 6351eb7 into alibaba:master Jan 4, 2021
@oldratlee oldratlee deleted the improve/maven-build branch January 4, 2021 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants