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.

Sign up for GitHub

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

Jump to bottom

default network is laos #597

Merged
merged 1 commit into from
May 17, 2024
Merged

default network is laos #597

merged 1 commit into from
May 17, 2024

Conversation

asiniscalchi
Copy link
Member

@asiniscalchi asiniscalchi commented May 16, 2024 •
edited by github-actions bot
Loading

PR Type

enhancement

Description

  • Updated the load_spec function to handle an empty string as an alias for the "laos" network, ensuring that the default network configuration points to "laos".

Changes walkthrough 📝

Relevant files
Enhancement
command.rs
Treat empty string as 'laos' network in load_spec function

node/src/command.rs

  • Modified the load_spec function to treat an empty string as an alias
    for the "laos" network.
    		• +1/-1     

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    @asiniscalchi
    Copy link
    Member Author

    /describe

    @github-actions GitHub Actions
    Copy link

    PR Description updated to latest commit (b528ff6)

    @asiniscalchi asiniscalchi merged commit 9ea2876 into main May 17, 2024
    8 checks passed
    @asiniscalchi asiniscalchi deleted the feature/laos_default_chain branch May 17, 2024 07:43
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Reviewers

    @ccubu ccubu ccubu approved these changes

    Assignees
    No one assigned
    Labels
    None yet
    Projects
    None yet
    Milestone
    No milestone
    Development

    Successfully merging this pull request may close these issues.
    2 participants
    @asiniscalchi @ccubu