-
Notifications
You must be signed in to change notification settings - Fork 31
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
Used gradle to init the project and package #1
Conversation
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.
Minor comments.
build.gradle
Outdated
|
||
dependencies { | ||
// This dependency is exported to consumers, that is to say found on their compile classpath. | ||
api 'org.apache.commons:commons-math3:3.6.1' |
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'm assuming we don't need this and was just the gradle 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.
Yes. It was auto-generated. I didn't change that as all the code was generated and was planning to remove it later. But I can do it now.
.idea | ||
*.iml | ||
.settings | ||
|
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.
There is a gradle-wrapper.jar below, I think we need *.jar here?
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.
The gradle-wrapper.jar is needed as it can help other developers to build the project without having Gradle installed on their machine.
settings.gradle
Outdated
* in the user manual at https://docs.gradle.org/6.5.1/userguide/multi_project_builds.html | ||
*/ | ||
|
||
rootProject.name = 'aws-embedded-metrics-java' |
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.
While the repo name is *-java, for classpathing and package naming we should drop the java suffix.
Code changes and unit tests for thread-safety
Description of changes:
Run
gradle init
to initialize the project.Project Name: aws-embedded-metrics-java
Package Name: software.amazon.awssdk.services.cloudwatchlogs.emf
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.