gitea icon indicating copy to clipboard operation
gitea copied to clipboard

Add support for the Friendly Forge Format (F3)

Open earl-warren opened this issue 3 years ago • 9 comments

This pull request is a mirror of a corresponding pull request on gitea.com, please go there to discuss and comment.


Continuation of this previous pull request

earl-warren avatar Sep 06 '22 05:09 earl-warren

How can it be that lint on windows reports an error and lint on GNU/Linux does not?

I think the only logical reason would be presence of //go:build tags that run only on Windows, but that doesn't seem to be the case here. Might just restart and see if it goes away.

silverwind avatar Sep 06 '22 09:09 silverwind

How can it be that lint on windows reports an error and lint on GNU/Linux does not?

I think the only logical reason would be presence of //go:build tags that run only on Windows, but that doesn't seem to be the case here. Might just restart and see if it goes away.

But there are no //go:build tags in this PR. Is there a way to re-launch the CI and verify this is not a false negative?

earl-warren avatar Sep 07 '22 00:09 earl-warren

Just push an empty commit to restart CI:

git commit --allow-empty -m 'restart ci'

silverwind avatar Sep 07 '22 11:09 silverwind

Now that's a new and interesting false negative.

image

earl-warren avatar Sep 08 '22 02:09 earl-warren

The CI is broken.

image

earl-warren avatar Sep 09 '22 00:09 earl-warren

We know. https://github.com/go-gitea/gitea/pull/21108

delvh avatar Sep 09 '22 07:09 delvh

Now that #21108 is merged, the CI is back to another false negative: failing lint on windows but not on GNU/Linux, despite the absence of build tags.

image

earl-warren avatar Sep 10 '22 02:09 earl-warren

Now that #21108 is merged, the CI is back to another false negative: failing lint on windows but not on GNU/Linux, despite the absence of build tags.

image

This pull request is not compiled for windows to workaround the linter false negative.

earl-warren avatar Sep 18 '22 16:09 earl-warren

The arm64 build crashes go 1.19

image

Because the gitea/test-env image for arm64 has 1.19 and not 1.19.1 which is missing a critical bug fix.

The test-env image must be rebuilt for arm64 and published.

earl-warren avatar Sep 21 '22 08:09 earl-warren

The arm64 test image was rebuilt with 1.19.1, trying again.

earl-warren avatar Sep 21 '22 14:09 earl-warren

Side note:

This pull request is a mirror of a corresponding pull request on gitea.com, please go there to discuss and comment.

gitea core development takes place on GitHub, so it should be reviewed/discussed here :)

silverwind avatar Sep 21 '22 19:09 silverwind

Side note:

This pull request is a mirror of a corresponding pull request on gitea.com, please go there to discuss and comment.

gitea core development takes place on GitHub, so it should be reviewed/discussed here :)

I did not want to force people to agree to the GitHub terms of service to participate in this particular pull request. Moving the discussion to gitea.com is more inclusive in a way that is not disruptive to the Gitea development process.

earl-warren avatar Sep 23 '22 07:09 earl-warren

Side note:

This pull request is a mirror of a corresponding pull request on gitea.com, please go there to discuss and comment.

gitea core development takes place on GitHub, so it should be reviewed/discussed here :)

I did not want to force people to agree to the GitHub terms of service to participate in this particular pull request. Moving the discussion to gitea.com is more inclusive in a way that is not disruptive to the Gitea development process.

I commented further on the issue you opened that is related to this topic.

earl-warren avatar Sep 24 '22 07:09 earl-warren

Just to mention this again, a valid point was raised on the previous PR for F3. I would like to state again that Loïc and the other contributors will change the license to MIT when this PR is approved to be merged.

Gusted avatar Sep 24 '22 15:09 Gusted

@earl-warren could you add me to your fork as contrib (write access)? so i can update that pull ... also if you would not force-push I would aprechiate!

6543 avatar Oct 16 '22 17:10 6543

blocked by https://lab.forgefriends.org/friendlyforgeformat/gof3/-/issues/21

6543 avatar Oct 16 '22 17:10 6543

please update branch & update f3 dep

6543 avatar Oct 24 '22 13:10 6543