Skip to content

Conversation

@JenabaBa
Copy link
Collaborator

This PR adds a script to update the dependencies in the geth, reth, and nethermind Dockerfiles without having to manually make changes to the files when the time comes to update.

In order to run the updater:

cd dependency_updater
go build dependency_updater.go
export GITHUB_TOKEN=<YOUR_TOKEN>
./dependency_updater

Running these commands will create a versions.env file which contains the most up to date versions of the dependencies and populate the Dockerfiles.

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jun 13, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@danyalprout
Copy link
Collaborator

This looks good, a bunch of small things to make the tool work with out other repos.

@danyalprout
Copy link
Collaborator

@JenabaBa another misc thought, lets setup some golang github actions for CI (happy for you to do it in another PR). You can use this job as a starting point.

dependencyPrefix, "REPO", dependencies[dependency].RepoUrl))
}

file, err := os.Create(repoPath + "/versions.env")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: you can use path.join to make this support multi-platforms in the future see: https://pkg.go.dev/path#Join

@danyalprout danyalprout self-requested a review June 17, 2025 20:05
@cb-heimdall
Copy link
Collaborator

Review Error for danyalprout @ 2025-06-17 20:07:20 UTC
User cannot review their own commit

@danyalprout danyalprout merged commit b10fc7a into base:main Jun 17, 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.

4 participants