Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent ckb-shared import ckb-db-schema #4524

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

eval-exec
Copy link
Collaborator

@eval-exec eval-exec commented Jul 17, 2024

What problem does this PR solve?

Problem Summary:

I believe that loading the COLUMN_NUMBER_HASH data in ckb-shared is not appropriate; it should be moved to ckb-store.

What is changed and how it works?

Related changes

  • Move loading the COLUMN_NUMBER_HASH data function from ckb-shared to ckb-store
  • Move loading the COLUMN_BLOCK_BODY data function from ckb-shared to ckb-store

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code ci-runs-only: [ quick_checks,linters ]

Side effects

  • None

Release note

Title Only: Include only the PR title in the release note.

@eval-exec eval-exec changed the title Move load COLUMN_NUMBER_HASH logic from ckb-shared to ckb-store Prevent ckb-shared import ckb-db-schema Jul 17, 2024
@eval-exec eval-exec changed the title Prevent ckb-shared import ckb-db-schema Prevent ckb-shared import ckb-db-schema and ckb-db Jul 17, 2024
@eval-exec eval-exec changed the title Prevent ckb-shared import ckb-db-schema and ckb-db Prevent ckb-shared import ckb-db-schema Jul 17, 2024
Copy link

github-actions bot commented Dec 5, 2024

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the stale To be closed due to a lack of activity label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale To be closed due to a lack of activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant