Skip to content

Fix build for #7711 after #7710 refactoring#7712

Merged
gregns1 merged 3 commits into
mainfrom
pr-7711-follow-up
Aug 28, 2025
Merged

Fix build for #7711 after #7710 refactoring#7712
gregns1 merged 3 commits into
mainfrom
pr-7711-follow-up

Conversation

@bbrks
Copy link
Copy Markdown
Member

@bbrks bbrks commented Aug 28, 2025

Integration Tests

  • n/a

@bbrks bbrks requested a review from gregns1 August 28, 2025 10:01
Copilot AI review requested due to automatic review settings August 28, 2025 10:01
Copy link
Copy Markdown
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 build issues that resulted from refactoring in #7710 by updating method calls and function signatures. The changes rename variables for clarity and modify a function to return additional type information for future usage.

  • Replace deprecated doc.CurrentRev calls with doc.GetRevTreeID() method calls
  • Rename and modify GetChangeEntryVersion() to GetChangeRowRev() with enhanced return type
  • Update caller to handle the new function signature

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
db/utilities_hlv_testing.go Updates method calls from CurrentRev field to GetRevTreeID() method
db/changes.go Renames function and modifies signature to return version type alongside version string
db/blip_handler.go Updates caller to handle new function signature with additional return value

@gregns1 gregns1 enabled auto-merge (squash) August 28, 2025 10:17
@gregns1 gregns1 merged commit 039ddef into main Aug 28, 2025
41 of 42 checks passed
@gregns1 gregns1 deleted the pr-7711-follow-up branch August 28, 2025 10:19
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