Skip to content

bump spec above mainnet#829

Merged
camfairchild merged 1 commit into
mainfrom
fix/lite-sync-issue
Sep 23, 2024
Merged

bump spec above mainnet#829
camfairchild merged 1 commit into
mainfrom
fix/lite-sync-issue

Conversation

@camfairchild

Copy link
Copy Markdown
Contributor

Description

This PR resolves an issue with lite nodes syncing the latest blocks. This is caused by a the spec_version being equal to the version live on mainnet.

We should add a CI-check for this before allowing merges to main. Or run an action to automatically bump the spec_version

Replaces #828

Related Issue(s)

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Other (please describe):

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have run cargo fmt and cargo clippy to ensure my code is formatted and linted correctly
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@camfairchild camfairchild merged commit 517fabf into main Sep 23, 2024
gztensor pushed a commit that referenced this pull request Nov 4, 2024
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.

Could not lookup information required to validate the transaction

3 participants