Skip to content

Drop deprecated methods#740

Merged
Z1kkurat merged 1 commit intomasterfrom
chore-drop-deprecations
Sep 2, 2025
Merged

Drop deprecated methods#740
Z1kkurat merged 1 commit intomasterfrom
chore-drop-deprecations

Conversation

@Z1kkurat
Copy link
Contributor

@Z1kkurat Z1kkurat commented Sep 2, 2025

No description provided.

@Z1kkurat Z1kkurat requested a review from Copilot September 2, 2025 12:00
Copy link

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 removes deprecated methods across multiple schema and key management classes to clean up the codebase. The deprecated methods were marked for removal since versions 5.0.6 and 6.1.3, and their removal supports API consolidation by forcing users to use newer method variants with explicit parameters.

Key changes:

  • Removes deprecated factory methods that lacked explicit table name parameters
  • Eliminates deprecated key management methods that didn't use proper logging prefixes
  • Consolidates API surface by removing redundant method overloads

Reviewed Changes

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

Show a summary per file
File Description
SnapshotSchema.scala Removes deprecated of method without explicit table name parameter
KeySchema.scala Removes deprecated of method without explicit table name parameter
JournalSchema.scala Removes deprecated of method without explicit table name parameter
KeysOf.scala Removes deprecated memory and apply methods lacking proper logging support
Keys.scala Removes deprecated apply method without LogPrefix requirement
KeyDatabase.scala Removes deprecated keysOf method without LogPrefix parameter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Z1kkurat Z1kkurat merged commit b44bfa7 into master Sep 2, 2025
5 checks passed
@Z1kkurat Z1kkurat deleted the chore-drop-deprecations branch September 2, 2025 12:31
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.

2 participants