Skip to content

Conversation

ziggie1984
Copy link
Collaborator

@ziggie1984 ziggie1984 commented Aug 14, 2025

remove expired key.

gemini-code-assist[bot]

This comment was marked as outdated.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes an expired PGP key, updates database migrations, and adds documentation for the SQL schema update process. The changes to remove the key are correct. The database migration changes introduce a new index for settled invoices and renumber the graph migrations. I've found a critical issue where development migration versions are not sequential, which could lead to database inconsistencies. I've also found a minor typo in the new documentation file. Please see my detailed comments.

@ziggie1984 ziggie1984 requested review from guggero and removed request for bitromortac August 15, 2025 06:53
@guggero guggero merged commit 365f178 into lightningnetwork:master Aug 15, 2025
23 of 24 checks passed
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.

3 participants