Skip to content

chore: update fetchup and got#1201

Open
secustor wants to merge 1 commit intogo-rod:mainfrom
secustor:chore/update-go-mods
Open

chore: update fetchup and got#1201
secustor wants to merge 1 commit intogo-rod:mainfrom
secustor:chore/update-go-mods

Conversation

@secustor
Copy link

@secustor secustor commented Jun 23, 2025

This PR updates github.com/ysmood/fetchup and github.com/ysmood/got to the newest version.

As visible in the first commit it bumped the version in go and adapted to changed APIs in fetchup

Development guide

Link

Test on local before making the PR

go run ./lib/utils/simple-check

The simple check script fails for changes outside of this PR so I could not run it outside of the full context.
That being said I can try to fix them, I have some local changes to fix most surface problems.

dir := lc.Dir()

fu := fetchup.New(dir, us...)
fu := fetchup.New(us...).WithSaveTo(dir)
Copy link
Author

Choose a reason for hiding this comment

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

@Fly-Playgroud Fly-Playgroud requested review from Fly-Playgroud and ysmood and removed request for ysmood June 30, 2025 14:11
@Fly-Playgroud Fly-Playgroud requested a review from ysmood July 1, 2025 08:24
@craig-stone-hpe
Copy link

I noticed this also and found @secustor already submitted a PR. Thank you very much :)
Pinning to the older github.com/ysmood/fetchup v0.3.0 until your PR is merged into a new release.

@ilovesusu
Copy link

Since the new version of fetchup has been adapted, why not merge it?

@rusq
Copy link

rusq commented Aug 10, 2025

@ysmood, could you look into merging this please? It's a pain to keep downgrading fetchup to v0.3.0 every time after go get -u ./... because rod@v116.2 still uses the old signature call.

@Fly-Playgroud Fly-Playgroud removed the request for review from ysmood August 13, 2025 10:00
@ryanbekhen
Copy link

why this, never merge?

@suntong
Copy link

suntong commented Dec 18, 2025

Review required
At least 1 approving review is required by reviewers with write access.

Yet,

Fly-Playgroud removed the request for review from ysmood 4 months ago

Hmm.. do the above mean that @ysmood can merge it now, or there need yet another reviewer?

@marwan-mustafa
Copy link

why this, never merge?

Very good question.. why?

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.

9 participants