-
Notifications
You must be signed in to change notification settings - Fork 9
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
Build scripts updated and refactored #225
Conversation
url = uri("https://plugins.gradle.org/m2") | ||
url = uri("http://dl.bintray.com/cognifide/maven-public") | ||
url = uri("https://dl.bintray.com/neva-dev/maven-public") | ||
} | ||
} | ||
|
||
dependencies { | ||
implementation("com.moowork.gradle:gradle-node-plugin:1.2.0") |
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.
Updating node causes a lot of problems... i left it un-updated
} | ||
|
||
dependencies { | ||
implementation("com.moowork.gradle:gradle-node-plugin:1.2.0") | ||
implementation("org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.31") | ||
implementation("com.bmuschko:gradle-docker-plugin:4.10.0") | ||
implementation("net.researchgate:gradle-release:2.6.0") |
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.
gradle-release
No longer in use
@@ -43,7 +43,7 @@ class BoardsAndWidgetsController : AbstractVerticle() { | |||
|
|||
private fun prepareConfig() = mapOf<String, (JsonObject) -> String>( | |||
"update" to { body -> update(body) }, | |||
"get" to { body -> boardsConfigService.loadBoardsConfig().toString() } | |||
"get" to { _ -> boardsConfigService.loadBoardsConfig().toString() } |
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.
Detekt fix
|
||
tasks { |
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.
task registration is now grouped in { ... }
tasks.register<Exec>("copyCypressContent") { | ||
group = "docker-functional-tests" | ||
commandLine = listOf("sh", "scripts/copyCypressContent.sh", "$cypressContentPath", "$mountDir") | ||
mustRunAfter("redeployLocal") |
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.
@staszke - We have better approach for copying files - lets use the old way without unnecessary script that will not work when you don't have shell sh
installed on your system.
maven { | ||
url = uri("https://plugins.gradle.org/m2") | ||
url = uri("http://dl.bintray.com/cognifide/maven-public") | ||
url = uri("https://dl.bintray.com/neva-dev/maven-public") | ||
} |
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.
lets group them together
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.
Nice
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.
Good job!
Just an update and refactor - no functional changes.
I hereby agree to the terms of the Cogboard Contributor License Agreement.