Skip to content

Gradle upgrade#115

Draft
soerendomroes wants to merge 8 commits into
mainfrom
chore/gradle7
Draft

Gradle upgrade#115
soerendomroes wants to merge 8 commits into
mainfrom
chore/gradle7

Conversation

@soerendomroes

Copy link
Copy Markdown
Member

No description provided.

// TODO help menus keep disappearing and is offset
menu(graphic = sochaIcon) {
item("Beenden", "Shortcut+Q").action {
logger.debug("Quitting!")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Why the random changes? Is logger.debug() deprecated?

Comment thread build.gradle.kts
arrayOf("year", "minor", "patch").map { versions["socha.version.$it"].toString().toInt() }.joinToString(".") + suffix
}

group = "sc.gui"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Why do you change this?

Comment thread build.gradle.kts
implementation(kotlin("reflect"))


implementation(files("./gradle/tornadofx2-21e933fd41.jar"))

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Why move this?

Comment thread build.gradle.kts
Comment on lines +116 to +117
)
// included because of tornadofx already

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I do not understand this change.

Comment thread README.md
Comment on lines -45 to -50
### Java-Versionen und Abhängigkeiten

Aktuell können die Backend-docs nur mit JDK 8 gebaut werden,
dieses Projekt braucht jedoch für [tornadofx](https://github.com/edvin/tornadofx2)
mindestens Java 11.
Daher müssen die Releases separat gebaut werden.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Could you readd the documentation about the current java version and limitations?

What Java version do I need?

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.

1 participant