feat(ci): overhaul CI/CD with smart detection & optimized caching #2101
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hubcio
force-pushed
the
fix-ci
branch
10 times, most recently
from
2a69d14 to
2741f3a
Compare
Contributor
lukaszzborek
left a comment
lukaszzborek
left a comment
There was a problem hiding this comment.
For me, in task 'test' should be run unit (Iggy_SDK_Tests) and integration tests (Iggy_SDK.Tests.Integration).
In 'e2e' only BDD (Iggy_SDK.Tests.BDD).
hubcio
force-pushed
the
fix-ci
branch
17 times, most recently
from
6832491 to
c0db658
Compare
hubcio
force-pushed
the
fix-ci
branch
10 times, most recently
from
3f28279 to
7c3c67a
Compare
hubcio
force-pushed
the
fix-ci
branch
5 times, most recently
from
f0163cb to
1f7c8cd
Compare
changhc
reviewed
Aug 29, 2025
Contributor
changhc
left a comment
changhc
left a comment
There was a problem hiding this comment.
nitpick: can we at least manually make sure that text files (i.e. non-binary files and files that do not reject additional characters) have a trailing newline character? I don't think there is any existing tool for this unfortunately and therefore this is hard to enforce, but I think it's a nice to have.
Contributor
Author
|
@changhc I added trailing newline check to _common.yml later, I will move all these into separate files like you suggested before, so that it'll be possible for devs to call them before commit / push locally. 
|
hubcio
force-pushed
the
fix-ci
branch
2 times, most recently
from
3b74900 to
942545b
Compare
Major improvements: - Intelligent change detection with dependency graph resolution - Only test affected components + transitive dependencies - Dual caching: sccache for artifacts + rust-cache for deps - Parallel matrix execution for all test suites - Python tests use server-start/stop instead of Docker - Centralized component config in components.yml - Unified test workflow handling all languages - Maven Nexus publishing for Java SDK - Multi-arch Docker builds with buildx - Manual tagging workflow for controlled releases - CI summary with execution times per component - License header & conventional commit validation Fixes: Missing Docker inputs, sccache persistence, order-sensitive pattern matching, empty duration column, Maven credentials. Reduces CI runtime by ~40% while improving reliability.
spetz
approved these changes
Aug 30, 2025
Contributor
|
Can't wait to see it in action 😀 |
numinnex
approved these changes
Aug 30, 2025
hageshiame
pushed a commit
to hageshiame/iggy
that referenced
this pull request
Nov 7, 2025
…ache#2101) Major improvements: - Intelligent change detection with dependency graph resolution - Only test affected components + transitive dependencies - Dual caching: sccache for artifacts + rust-cache for deps - Parallel matrix execution for all test suites - Python tests use server-start/stop instead of Docker - Centralized component config in components.yml - Unified test workflow handling all languages - Maven Nexus publishing for Java SDK - Multi-arch Docker builds with buildx - Manual tagging workflow for controlled releases - CI summary with execution times per component - License header & conventional commit validation Fixes: Missing Docker inputs, sccache persistence, order-sensitive pattern matching, empty duration column, Maven credentials. Reduces CI runtime by ~40% while improving reliability.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Major improvements:
Fixes: Missing Docker inputs, sccache persistence, order-sensitive
pattern matching, empty duration column, Maven credentials.
Reduces CI runtime by ~40% while improving reliability.