Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 14, 2025

This PR adds the required minimumDappnodeVersion field to the Gnosis variant configuration as part of standardizing DAppNode package requirements across all variants.

Changes Made

  • Updated existing /package_variants/gnosis/dappnode_package.json with minimumDappnodeVersion: "0.3.11"
  • Added the minimum version requirement at the root level of the JSON structure
  • Preserved all existing Gnosis variant configuration (name, version, links, exposable endpoints)

Context

For generic DAppNode packages, variant-specific configurations need to specify the minimum DAppNode version required for proper functionality. This ensures compatibility and prevents deployment issues on older DAppNode versions.

The Gnosis variant now explicitly requires DAppNode version 0.3.11 or higher, which aligns with the package's dependency requirements and feature usage.

Validation

  • JSON syntax validated with jq
  • File structure matches existing variant patterns
  • All original configuration preserved
Original prompt

Add or update the following field in the appropriate dappnode_package.json file:

"minimumDappnodeVersion": "0.3.11"

Instructions:

  • If the repository is of type "generic" (as per its name), update the file variants/gnosis/dappnode_package.json (create the file if it does not exist).
  • If the minimumDappnodeVersion field already exists, update it to "0.3.11".
  • If not present, add the field at the root level of the JSON.
  • If the repository is not of type "generic", update dappnode_package.json at the root (create it if it does not exist) with the same logic.
  • Ensure the JSON remains valid after the change.

This pull request was created as a result of the following prompt from Copilot chat.

Add or update the following field in the appropriate dappnode_package.json file:

"minimumDappnodeVersion": "0.3.11"

Instructions:

  • If the repository is of type "generic" (as per its name), update the file variants/gnosis/dappnode_package.json (create the file if it does not exist).
  • If the minimumDappnodeVersion field already exists, update it to "0.3.11".
  • If not present, add the field at the root level of the JSON.
  • If the repository is not of type "generic", update dappnode_package.json at the root (create it if it does not exist) with the same logic.
  • Ensure the JSON remains valid after the change.

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: pablomendezroyo <41727368+pablomendezroyo@users.noreply.github.com>
Copilot AI changed the title [WIP] Update minimumDappnodeVersion in dappnode_package.json Add minimumDappnodeVersion field to variants/gnosis/dappnode_package.json Oct 14, 2025
Copilot AI requested a review from pablomendezroyo October 14, 2025 10:36
@pablomendezroyo
Copy link
Contributor

@copilot modify the existing file from directory package_variants instead of creating new one.

…ing new variants directory

Co-authored-by: pablomendezroyo <41727368+pablomendezroyo@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Oct 14, 2025

@copilot modify the existing file from directory package_variants instead of creating new one.

Updated the existing package_variants/gnosis/dappnode_package.json file instead of creating a new variants directory. The minimumDappnodeVersion: "0.3.11" field has been added to the existing file structure. Commit: 3b8c472

Copilot AI changed the title Add minimumDappnodeVersion field to variants/gnosis/dappnode_package.json Add minimumDappnodeVersion field to package_variants/gnosis/dappnode_package.json Oct 14, 2025
@github-actions
Copy link
Contributor

Dappnode bot has built and pinned the built packages to an IPFS node, for commit: 2113bd7

This is a development version and should only be installed for testing purposes.

  1. Package erigon.dnp.dappnode.eth

Install link

Hash: /ipfs/QmVyu2u9v3dAwqXVMG7JYdVRVu1LA7RKArJiDPQeEAtXkH

(by dappnodebot/build-action)

@pablomendezroyo pablomendezroyo marked this pull request as ready for review October 14, 2025 10:47
@pablomendezroyo pablomendezroyo merged commit 5646222 into main Oct 14, 2025
7 checks passed
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