-
Notifications
You must be signed in to change notification settings - Fork 13
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
micronaut framework implemented. #673
base: main
Are you sure you want to change the base?
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.
Thanks for the PR!
This PR does not build and compile. Could you please apply the changes, and the open the project from settings.gradle.kts
level. I observe gradle-wrapper
additions which should not be case for any change.
Also instead of micronaut-starter, lets keep micronaut
as the name for the folder.
id("stove-publishing") apply false | ||
idea | ||
java | ||
} | ||
group = "com.trendyol" | ||
version = CI.version(project) | ||
version = version() |
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 wouldn't expect any change for this line.
dependencies { | ||
related.forEach { | ||
kover(it) | ||
} | ||
} | ||
|
||
subprojects.of("lib", "spring", "examples", "ktor") { | ||
subprojects.of("lib", "spring", "examples", "ktor", "micronaut-starter") { |
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.
micronaut
is better naming instead of starter.
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 shouldn't be any gradle-wrapper extra. Why this build system added?
} | ||
|
||
dependencies { | ||
runtimeOnly("org.yaml:snakeyaml:2.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.
We don't add hardcoded versions, they need to be moved into libs.versions.toml.
id("io.micronaut.aot") version "4.4.3" | ||
} | ||
|
||
repositories { |
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 should not be repository definition here.
kotlin("plugin.serialization") version libs.versions.kotlin | ||
application | ||
idea | ||
id("com.google.devtools.ksp") version "1.9.25-1.0.20" |
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 don't use hardcoded.
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 also a mistake probably.
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.
Please remove .idea
folders.
@@ -42,21 +40,41 @@ kover { | |||
} | |||
} | |||
} | |||
val related = subprojects.of("lib", "spring", "examples", "ktor") | |||
val related = subprojects.of("lib", "spring", "examples", "ktor", "micronaut-starter") |
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.
micronaut
plugin(rootProject.libs.plugins.testLogger.get().pluginId) | ||
plugin(rootProject.libs.plugins.kover.get().pluginId) | ||
plugin(rootProject.libs.plugins.detekt.get().pluginId) | ||
plugin( |
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.
Could you revert the alignment back.
) | ||
|
||
subprojects.of("lib", "spring", "ktor", filter = { p -> publishedProjects.contains(p.name) }) { | ||
subprojects.of("lib", "spring", "ktor", "micronaut-starter", filter = { p -> publishedProjects.contains(p.name) }) { |
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.
micronaut
@@ -160,3 +178,11 @@ tasks.withType<DokkaMultiModuleTask>().configureEach { | |||
outputDirectory.set(file(rootDir.resolve("docs/source"))) | |||
} | |||
|
|||
fun version(): String = when { |
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 was deleted. It shouldn't be here.
@@ -0,0 +1,13 @@ | |||
dependencies { | |||
api(projects.lib.stoveTestingE2e) | |||
implementation("io.micronaut:micronaut-core") // Temel Micronaut bağımlılığı |
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.
- Remove comment
- Do not use hardcoded versions
testImplementation("io.kotest:kotest-runner-junit5:5.7.2") | ||
} | ||
|
||
repositories { |
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.
not necessary
@JvmStatic fun main(args: Array<String>) { | ||
Micronaut.build() | ||
.args(*args) | ||
.packages("your.base.package.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.
why do we have hardcoded package name?
} | ||
|
||
override suspend fun afterProject() { | ||
// Uygulama durdurma işlemi yapılabilir |
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.
remove the comment.
micronaut example created
micronaut starter created