Skip to content

chore: add marshalByValue comment for generation in highlight service#2246

Merged
rkaraivanov merged 1 commit into
masterfrom
mdragnev/update-highlight-metadata
Jun 8, 2026
Merged

chore: add marshalByValue comment for generation in highlight service#2246
rkaraivanov merged 1 commit into
masterfrom
mdragnev/update-highlight-metadata

Conversation

@mddragnev

Copy link
Copy Markdown
Member

Description

add marshalByValue comment for generation in highlight service

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that causes existing functionality to change)
  • Documentation update
  • Refactoring (code improvements without functional changes)

Related Issues

Closes #

Testing

Checklist

  • My code follows the project's coding standards
  • I have tested my changes locally
  • I have updated documentation if needed
  • Breaking changes are documented in the description

@mddragnev mddragnev requested a review from rkaraivanov June 8, 2026 10:37
Copilot AI review requested due to automatic review settings June 8, 2026 10:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the highlight component’s service type metadata by adding a /* marshalByValue */ analyzer directive comment, likely to influence downstream API/code generation behavior for HighlightNavigation.

Changes:

  • Added /* marshalByValue */ comment alongside the existing /* jsonAPIPlainObject */ marker in the highlight service.

@rkaraivanov rkaraivanov merged commit 7dc2f90 into master Jun 8, 2026
8 checks passed
@rkaraivanov rkaraivanov deleted the mdragnev/update-highlight-metadata branch June 8, 2026 10:41
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.

4 participants