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

micronaut framework implemented. #673

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

anilcangul
Copy link

micronaut example created
micronaut starter created

Copy link
Collaborator

@osoykan osoykan left a 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()
Copy link
Collaborator

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") {
Copy link
Collaborator

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.

Copy link
Collaborator

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")
Copy link
Collaborator

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 {
Copy link
Collaborator

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"
Copy link
Collaborator

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.

Copy link
Collaborator

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.

Copy link
Collaborator

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")
Copy link
Collaborator

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(
Copy link
Collaborator

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) }) {
Copy link
Collaborator

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 {
Copy link
Collaborator

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ığı
Copy link
Collaborator

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 {
Copy link
Collaborator

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")
Copy link
Collaborator

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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the comment.

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