Skip to content

Modernize go packages #135

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

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

Modernize go packages #135

wants to merge 3 commits into from

Conversation

hamiltop
Copy link
Member

@hamiltop hamiltop commented Apr 9, 2025

Authored by openhands + claude sonnet 3.7 (via bedrock)

Initial instructions:

This is a set of go packages that are used in a bunch of microservices. Please modernize all these packages.

Total cost: $32.2275

Interventions:

It kept trying to use golang 1.21.3 instead of the latest golang

The latest stable version of go is 1.24.0 Let's switch to that version and continue

It kept checking files in a loop

Let's keep track of files we need to check in a file called modernize.md and then mark them as complete once we've checked them so we don't keep rechecking them.

It checked modernize.md but wasn't updating it

Let's record which files we've already checked

More loops

Please remember to update modernize.md

More loops

Let's focus on actually making changes. We keep checking files but after we check a file we should either update that file or we should update modernize.md to reflect that we don't need to make any changes to that file.

Wrap up

Let's commit this to git. Let's do all the vendor updates in an initial single commit and then all the rest of the changes in semantically helpful individual commits.
That took a lot of work. 503 steps. It cost over $30. A lot of that seemed to be spinning in circles and I had to interrupt and get you back on track.

Please write up a set of instructions for modernizing a go repo like this one using openhands that we can reference the next time we need to modernize a go repo. Include instructions for installing go, using a file to track work in progress, the types of updates made, etc.

Save those instructions into a file called openhands.instructions.md.
Commit the rest of our files and let's push to github

Full transcript:

openhands_remind101_pkg_modernize.json

@hamiltop hamiltop force-pushed the modernize-go-packages branch from 0596763 to a450813 Compare April 9, 2025 01:49
@hamiltop
Copy link
Member Author

hamiltop commented Apr 9, 2025

A few follow on commits to handle a minor update to 1.24.2.

@hamiltop hamiltop force-pushed the modernize-go-packages branch from a450813 to fd4b1d1 Compare April 9, 2025 14:03
@hamiltop
Copy link
Member Author

hamiltop commented Apr 9, 2025

Main commit: fd4b1d1

Copy link

@franleplant franleplant left a comment

Choose a reason for hiding this comment

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

i can hardly approve this PR due to the UI being buggy due to the large PR

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