AGit based PRs fail #23

New issue

Open

opened 2024-07-02 07:29:09 +00:00 by Ghost · 1 comment

Ghost commented

2024-07-02 07:29:09 +00:00

Discovered in: forgejo/act#42

Discovered in: https://code.forgejo.org/forgejo/act/pulls/42

👍 2

earl-warren added the

Kind/Bug

label

2025-07-20 08:58:12 +00:00

earl-warren commented

2025-08-18 10:46:30 +00:00

Contributor

A note on AGit pull requests and go modules.

go: downloading code.forgejo.org/earl-warren/runner/v9 v9.0.0-20250818095807-8a95925a970d
go: downloading github.com/alecthomas/repr v0.5.1

that works fine because the SHA is reachable from a branch, however...

go: downloading code.forgejo.org/forgejo/runner/v9 v9.0.0-20250818095807-8a95925a970d
go: forgejo.org/models/actions imports
	code.forgejo.org/forgejo/runner/v9/act/jobparser: code.forgejo.org/forgejo/runner/v9@v9.0.0-20250818095807-8a95925a970d: invalid version: unknown revision 8a95925a970d

because the SHA is not reachable via any reference (heads or tags) but only from refs/pull/N/head which could be discovered if explicitly asking for all references to be fetched, not just refs/heads/* and refs/tags/*
am I understanding that correctly or did I miss a turn?

A note on AGit pull requests and go modules. ``` go: downloading code.forgejo.org/earl-warren/runner/v9 v9.0.0-20250818095807-8a95925a970d go: downloading github.com/alecthomas/repr v0.5.1 ``` that works fine because the SHA is reachable from a branch, however... ``` go: downloading code.forgejo.org/forgejo/runner/v9 v9.0.0-20250818095807-8a95925a970d go: forgejo.org/models/actions imports code.forgejo.org/forgejo/runner/v9/act/jobparser: code.forgejo.org/forgejo/runner/v9@v9.0.0-20250818095807-8a95925a970d: invalid version: unknown revision 8a95925a970d ``` because the SHA is not reachable via any reference (heads or tags) but only from refs/pull/N/head which could be discovered if explicitly asking for all references to be fetched, not just refs/heads/* and refs/tags/* am I understanding that correctly or did I miss a turn?

earl-warren changed title from ~~Agit based PRs fail~~ to AGit based PRs fail

2025-08-18 10:46:40 +00:00