Skip to content

Commit

Permalink
chore: EXPOSED-573 Project detekt config does not check .kt or .kts i…
Browse files Browse the repository at this point in the history
…n docs

- Local detekt workflow does not check documentation files, whereas the GitHub
workflow checks them if they include changes.
- Add documentation source files to project detekt config, so that results match
on TC and when PR opened.
  • Loading branch information
bog-walk committed Jan 22, 2025
1 parent 4c52764 commit f65e51f
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,13 @@ fun Project.configureDetekt() {
apply<DetektPlugin>()

configure<DetektExtension> {
// source.from(
// files(
// "$rootDir/documentation-website/Writerside/snippets",
// "$rootDir/samples"
// )
// )

ignoreFailures = false
buildUponDefaultConfig = true
parallel = true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import kotlinx.datetime.LocalDateTime
import org.example.tables.BaseTable
import org.jetbrains.exposed.dao.IntEntity
import org.jetbrains.exposed.dao.id.EntityID
// import org.jetbrains.exposed.dao.id.IntIdTable

abstract class BaseEntity(id: EntityID<Int>, table: BaseTable) : IntEntity(id) {
val created: LocalDateTime by table.created
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import org.jetbrains.exposed.sql.tests.shared.dml.withCitiesAndUsers
import org.jetbrains.exposed.sql.tests.shared.entities.EntityTestsData
import org.junit.Test
import java.math.BigDecimal
import java.util.UUID
import kotlin.test.assertEquals
import kotlin.test.assertNotNull
import kotlin.test.assertNull
Expand Down

0 comments on commit f65e51f

Please sign in to comment.