Skip to content

Conversation

@alexander-yevsyukov
Copy link
Collaborator

This PR fixes the documentation links in the ErrorPlaceholder class which could not be resolved by Dokka because the context modules does not have dependency on the runtime module. This caused the error when publishing.

Other notable changes

  • Added io.spine.validate.FieldAwareMessage interface for backward compatibility during the transition to io.spine.validation. The absence of the interface cased strict warnings during compilation of stubs in Kotlin.

@alexander-yevsyukov alexander-yevsyukov requested review from Copilot and removed request for Copilot December 19, 2025 15:33
@alexander-yevsyukov alexander-yevsyukov marked this pull request as ready for review December 19, 2025 15:33
Copilot AI review requested due to automatic review settings December 19, 2025 15:33
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes broken KDoc links in the ErrorPlaceholder class that prevented successful documentation publishing with Dokka. The links could not be resolved because the context module lacks a dependency on the runtime module.

Key changes:

  • Removed unresolvable KDoc @see references to runtime module classes
  • Added backward compatibility interface for FieldAwareMessage to prevent Kotlin compilation warnings
  • Updated version to 2.0.0-SNAPSHOT.381

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
context/src/main/kotlin/io/spine/tools/validation/ErrorPlaceholder.kt Removed @see tags referencing classes in the runtime module to fix broken KDoc links
jvm-runtime/src/main/java/io/spine/validate/FieldAwareMessage.java Added deprecated interface for backward compatibility during migration to io.spine.validation
version.gradle.kts Version bump to 2.0.0-SNAPSHOT.381
pom.xml Version bump to 2.0.0-SNAPSHOT.381
dependencies.md Auto-generated dependency report with updated timestamps and version references
config Updated subproject commit reference

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.92%. Comparing base (ab54f67) to head (f7708d8).
⚠️ Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #254   +/-   ##
=========================================
  Coverage     34.92%   34.92%           
  Complexity      276      276           
=========================================
  Files           137      137           
  Lines          3055     3055           
  Branches        244      244           
=========================================
  Hits           1067     1067           
  Misses         1914     1914           
  Partials         74       74           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alexander-yevsyukov alexander-yevsyukov self-assigned this Dec 19, 2025
Copy link
Collaborator

@armiol armiol left a comment

Choose a reason for hiding this comment

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

LGTM.
JFYI.

Previously, I have reproduced the past error locally, and now I have checked the publishing again, and it works fine.

@alexander-yevsyukov alexander-yevsyukov merged commit 563d514 into master Dec 19, 2025
7 checks passed
@alexander-yevsyukov alexander-yevsyukov deleted the trigger-publishing branch December 19, 2025 16:06
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.

3 participants