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

Add testnet4 entry in chainparams #7894

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

2seaq
Copy link

@2seaq 2seaq commented Dec 2, 2024

Added an entry in chainparams for testnet4

Changelog-Added: entry in chainparams for testnet4

Important

24.11 FREEZE NOVEMBER 7TH: Non-bugfix PRs not ready by this date will wait for 25.02.

Checklist

Before submitting the PR, ensure the following tasks are completed. If an item is not applicable to your PR, please mark it as checked:

  • The changelog has been updated in the relevant commit(s) according to the guidelines.
  • Tests have been added or modified to reflect the changes.
  • Documentation has been reviewed and updated as needed.
  • Related issues have been listed and linked, including any that this PR closes.

@niftynei
Copy link
Collaborator

niftynei commented Dec 3, 2024

just as a driveby review comment: we don't use merge commits in CLN (your last commit is a merge commit). we use rebases. you'll need to remove that commit before this PR is eligible for being included in the main branch :)

@cdecker cdecker force-pushed the add-testnet4-support branch from 8e98ad3 to ebf3e84 Compare December 6, 2024 10:57
@cdecker
Copy link
Member

cdecker commented Dec 6, 2024

Rebased on top of master in order to remove the merge commits. I also squashed the two fixups that you had.

@cdecker cdecker self-assigned this Dec 6, 2024
@cdecker
Copy link
Member

cdecker commented Dec 6, 2024

cACK

@cdecker
Copy link
Member

cdecker commented Dec 6, 2024

Ah, I hate the Update PR button, it defaults to merge commits...

… testnet4 to correspond with chaininfo value.
@cdecker cdecker force-pushed the add-testnet4-support branch from 28de1b3 to 1b58069 Compare December 6, 2024 11:27
@2seaq
Copy link
Author

2seaq commented Dec 11, 2024

eek sorry. I should have checked first

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.

3 participants