Skip to content

A0-0000: Don't import unimportable blocks#1959

Merged
Marcin-Radecki merged 1 commit intoCardinal-Cryptography:mainfrom
timorleph:soliticors
Mar 21, 2025
Merged

A0-0000: Don't import unimportable blocks#1959
Marcin-Radecki merged 1 commit intoCardinal-Cryptography:mainfrom
timorleph:soliticors

Conversation

@timorleph
Copy link
Contributor

Description

Our sync tried importing everything that was required. This was not a major issue before 15.1, since at worst the import failed and whatever, but with optimistic imports we gotta be relatively confident that the import will actually work.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have added tests
  • I have made corresponding changes to the existing documentation
  • I have created new documentation

@Marcin-Radecki Marcin-Radecki added this pull request to the merge queue Mar 21, 2025
Merged via the queue into Cardinal-Cryptography:main with commit d36c3a5 Mar 21, 2025
14 of 15 checks passed
timorleph added a commit to timorleph/aleph-node that referenced this pull request Mar 21, 2025
# Description

Our sync tried importing everything that was required. This was not a
major issue before 15.1, since at worst the import failed and whatever,
but with optimistic imports we gotta be relatively confident that the
import will actually work.

## Type of change

- Bug fix (non-breaking change which fixes an issue)

# Checklist:

- I have added tests
- I have made corresponding changes to the existing documentation
- I have created new documentation
Marcin-Radecki pushed a commit that referenced this pull request Mar 21, 2025
# Description

Our sync tried importing everything that was required. This was not a
major issue before 15.1, since at worst the import failed and whatever,
but with optimistic imports we gotta be relatively confident that the
import will actually work.

## Type of change

- Bug fix (non-breaking change which fixes an issue)

# Checklist:

- I have added tests
- I have made corresponding changes to the existing documentation
- I have created new documentation
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