Skip to content

Conversation

@fluentdo-ci
Copy link
Contributor

@fluentdo-ci fluentdo-ci commented Feb 9, 2026

Update version


Summary by cubic

Bump Fluentdo Agent version from 25.10.16 to 25.10.17 across Dockerfiles, build scripts (CMake, setup-code.sh), and test scripts to publish v25.10.17 LTS artifacts. No functional changes; this keeps images and tests aligned with the release.

Written for commit e8c93e3. Summary will update on new commits.

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 6 files

Confidence score: 3/5

  • Missing test updates despite changes in source/ suggests unverified behavior, which raises regression risk
  • This is a concrete process gap (severity 7/10) rather than a proven bug, so merge risk is moderate but not blocking
  • Pay close attention to source/CMakeLists.txt - production changes without corresponding tests in source/tests/ or source/testing/.
Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="source/CMakeLists.txt">

<violation number="1" location="source/CMakeLists.txt:11">
P1: Rule violated: **Ensure tests are added**

This PR changes a production source file under source/ but includes no test updates under source/tests/ or source/testing/, violating the “Ensure tests are added” rule. Add or update an automated test file in source/tests/ or source/testing/ to cover the change.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

set(FLB_VERSION_MAJOR 25)
set(FLB_VERSION_MINOR 10)
set(FLB_VERSION_PATCH 16)
set(FLB_VERSION_PATCH 17)
Copy link

@cubic-dev-ai cubic-dev-ai bot Feb 9, 2026

Choose a reason for hiding this comment

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

P1: Rule violated: Ensure tests are added

This PR changes a production source file under source/ but includes no test updates under source/tests/ or source/testing/, violating the “Ensure tests are added” rule. Add or update an automated test file in source/tests/ or source/testing/ to cover the change.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At source/CMakeLists.txt, line 11:

<comment>This PR changes a production source file under source/ but includes no test updates under source/tests/ or source/testing/, violating the “Ensure tests are added” rule. Add or update an automated test file in source/tests/ or source/testing/ to cover the change.</comment>

<file context>
@@ -8,7 +8,7 @@ set(CMAKE_POLICY_DEFAULT_CMP0069 NEW)
 set(FLB_VERSION_MAJOR 25)
 set(FLB_VERSION_MINOR 10)
-set(FLB_VERSION_PATCH 16)
+set(FLB_VERSION_PATCH 17)
 set(FLB_VERSION_STR "${FLB_VERSION_MAJOR}.${FLB_VERSION_MINOR}.${FLB_VERSION_PATCH}")
 
</file context>
Fix with Cubic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant