Skip to content
This repository was archived by the owner on Jul 6, 2026. It is now read-only.

Develop#25

Merged
buijs-dev merged 28 commits into
mainfrom
develop
Nov 14, 2023
Merged

Develop#25
buijs-dev merged 28 commits into
mainfrom
develop

Conversation

@buijs-dev

Copy link
Copy Markdown
Owner

No description provided.

buijs-dev and others added 12 commits October 12, 2023 21:46
…p args and retrieve it everywhere flutter is used internally. TODO: use kommand cli to execute flutter/gradle commands.
…o not serialize duplicate dependencies in klutter.yaml. [3] Bugfix: incorrect keyword in generated IosAdapter. [4] Bump klutter (pub) and klutter_ui (pub) dependency versions.
…p args and retrieve it everywhere flutter is used internally. TODO: use kommand cli to execute flutter/gradle commands.
# Conflicts:
#	CHANGELOG.md
#	gradle/wrapper/gradle-wrapper.jar
#	gradlew
#	lib-build/src/main/resources/publish.properties
#	lib/annotations/annotations.podspec
#	lib/gradle/src/main/resources/publish.properties
#	lib/kore/src/main/kotlin/dev/buijs/klutter/kore/project/PubspecBuilder.kt
#	lib/tasks/src/main/kotlin/dev/buijs/klutter/tasks/project/ActionInitKlutter.kt
@codecov

codecov Bot commented Oct 13, 2023

Copy link
Copy Markdown

Codecov Report

Merging #25 (7915d92) into main (d8e6f1c) will decrease coverage by 7.84%.
Report is 1 commits behind head on main.
The diff coverage is 41.39%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #25      +/-   ##
============================================
- Coverage     78.83%   71.00%   -7.84%     
- Complexity      202      212      +10     
============================================
  Files            83      101      +18     
  Lines          2244     2700     +456     
  Branches        412      469      +57     
============================================
+ Hits           1769     1917     +148     
- Misses          363      657     +294     
- Partials        112      126      +14     
Files Coverage Δ
.../dev/buijs/klutter/compiler/processor/Processor.kt 50.00% <ø> (ø)
...js/klutter/compiler/processor/ProcessorProvider.kt 100.00% <100.00%> (ø)
...uijs/klutter/compiler/scanner/ControllerResults.kt 100.00% <100.00%> (+3.57%) ⬆️
...in/dev/buijs/klutter/gradle/dsl/DependencyUtils.kt 62.50% <ø> (ø)
...n/dev/buijs/klutter/gradle/dsl/KlutterExtension.kt 47.36% <ø> (+4.51%) ⬆️
...lin/dev/buijs/klutter/gradle/dsl/KlutterVersion.kt 90.47% <ø> (-0.83%) ⬇️
...kotlin/dev/buijs/klutter/gradle/tasks/CopyTasks.kt 22.22% <ø> (ø)
...rc/main/kotlin/dev/buijs/klutter/kore/Exception.kt 100.00% <100.00%> (ø)
...n/kotlin/dev/buijs/klutter/kore/project/Android.kt 90.00% <ø> (-3.34%) ⬇️
...in/kotlin/dev/buijs/klutter/kore/project/Config.kt 100.00% <100.00%> (ø)
... and 52 more

... and 1 file with indirect coverage changes

Impacted file tree graph

Comment thread gradlew

enum class OperatingSystem(val value: String) {
WINDOWS("windows"),
MACOS("macos")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

don't forget linux

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Good point, thanks! Need to test on Linux to.

Remove packages in kommand module.
Add Linux to OS checking.
TODO: Kommand options (add more interactive options).
TODO: Test Jetbrains plugin.
Add Tasks to download kradle.
Update commands to use klutter-dart 2.0.0.
Move tasks to kore package.
Rename kommand to kradle.
… build creates platform independent distributions.
Add flutter and gradlew executable commands.
Add build command to build default platform module.
…urces or fix gradle dependency between kradle and gradle modules.
@buijs-dev
buijs-dev merged commit efbdc9c into main Nov 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants