Skip to content

Conversation

@RaveenaBhasin
Copy link
Member

@RaveenaBhasin RaveenaBhasin commented May 30, 2025

User description

Reverts #17


PR Type

Bug fix


Description

  • Restore include_frontend parameter in Blockscout configuration

  • Update default Blockscout frontend image version

  • Revert Blockscout package import to previous state

  • Ensure parameter propagation across input parsing and sanity checks


Changes walkthrough 📝

Relevant files
Bug fix
main.star
Restore Blockscout import and include_frontend param usage

main.star

  • Revert Blockscout package import path to previous version
  • Add include_frontend to Blockscout general_args dictionary
  • +2/-1     
    input_parser.star
    Restore and set default for include_frontend in input parsing

    src/package_io/input_parser.star

  • Add include_frontend to blockscout_params struct in input_parser
  • Set default value for include_frontend in
    get_default_blockscout_params
  • Update default frontend image version for Blockscout
  • +3/-1     
    sanity_check.star
    Add include_frontend to Blockscout sanity check params     

    src/package_io/sanity_check.star

    • Add include_frontend to blockscout_params required fields
    +1/-0     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @qodo-code-review
    Copy link

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-code-review
    Copy link

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @RaveenaBhasin RaveenaBhasin merged commit eb1b92a into main May 30, 2025
    8 of 15 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants