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

Iron - v2.0.0 release #21

Open
wants to merge 22 commits into
base: santio/feat/controllers
Choose a base branch
from

Conversation

SantioMC
Copy link
Member

@SantioMC SantioMC commented Sep 26, 2024

This pull request will overview everything happening in the 2.0 release of Iron. Major changes include:

  • Making models compile-time to assist with performance
  • Moving to wrapping over Hikari for pooling
  • Cleaning up a lot of duplicate code
  • Introducing controllers

This pull request introduces breaking changes.

@SantioMC SantioMC changed the base branch from master to santio/feat/controllers September 26, 2024 03:18
@SantioMC SantioMC changed the title santio/fix/collections santio/refactor/2.0-cleanup Sep 26, 2024
@DebitCardz DebitCardz self-requested a review September 27, 2024 07:01
Comment on lines 25 to 27
test-sqlite = { module = "org.xerial:sqlite-jdbc", version.ref = "sqlite" }
test-log4j = { module = "org.apache.logging.log4j:log4j-core", version.ref = "log4j" }
test-log4j-impl = { module = "org.apache.logging.log4j:log4j-slf4j2-impl", version.ref = "log4j" }
test-h2 = { module = "com.h2database:h2", version.ref = "h2" }
test-logback = { module = "ch.qos.logback:logback-classic", version.ref = "logback" }
Copy link
Member

Choose a reason for hiding this comment

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

why're we prefixing with test-, just let it be h2, logback etc. and we import as a test impl in our gradle.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is to prevent accidental usage of these dependencies in non-test environments by providing them a label

src/main/kotlin/gg/ingot/iron/model/ModelReader.kt Outdated Show resolved Hide resolved
src/main/kotlin/gg/ingot/iron/model/ModelReader.kt Outdated Show resolved Hide resolved
src/main/kotlin/gg/ingot/iron/model/ModelReader.kt Outdated Show resolved Hide resolved
@SantioMC SantioMC changed the title santio/refactor/2.0-cleanup Iron - v2.0.0 release Oct 12, 2024
@SantioMC SantioMC self-assigned this Oct 12, 2024
@SantioMC SantioMC added the enhancement New feature or request label Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants