Skip to content

Conversation

@strk
Copy link
Member

@strk strk commented Nov 20, 2025

That hook is the one required for AGit workflow, or clients will get:

remote: error: cannot find hook 'proc-receive'

For the record: I tried creating this PR using AGit but GitHub doesn't suppor it... how long should we wait before Gitea development is moved to a Gitea instance ?

That hook is the one required for AGit workflow, or clients will get:

  remote: error: cannot find hook 'proc-receive'
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 20, 2025
@github-actions github-actions bot added modifies/translation docs-update-needed The document needs to be updated synchronously labels Nov 20, 2025
@silverwind
Copy link
Member

For the record: I tried creating this PR using AGit but GitHub doesn't suppor it... how long should we wait before Gitea development is moved to a Gitea instance ?

Last I checked, gitea.com was not performing as well as github.com at least for european users, it often had multi-second page load times, so I refrained from using it. Also remember the barrier of entry for new users is much lower on github.com where everyone already has an account.

@lunny
Copy link
Member

lunny commented Nov 20, 2025

For the record: I tried creating this PR using AGit but GitHub doesn't suppor it... how long should we wait before Gitea development is moved to a Gitea instance ?

Last I checked, gitea.com was not performing as well as github.com at least for european users, it often had multi-second page load times, so I refrained from using it. Also remember the barrier of entry for new users is much lower on github.com where everyone already has an account.

Could you evaluate it again now? I believe the performance should be fast enough at this point. Since gitea.com already supports logging in directly with GitHub OAuth2, I think it’s a good time to proceed with the migration.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 20, 2025
@lunny lunny added this to the 1.26.0 milestone Nov 20, 2025
@wxiaoguang
Copy link
Contributor

Since gitea.com already supports logging in directly with GitHub OAuth2, I think it’s a good time to proceed with the migration.

It is not able to remember the login, makes it very annoying to use.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 21, 2025
@techknowlogick techknowlogick merged commit 543e3bf into go-gitea:main Nov 21, 2025
25 checks passed
@lunny
Copy link
Member

lunny commented Nov 21, 2025

Since gitea.com already supports logging in directly with GitHub OAuth2, I think it’s a good time to proceed with the migration.

It is not able to remember the login, makes it very annoying to use.

Which service is failing to remember the login — Gitea.com or GitHub’s OAuth2? I’m unable to reproduce the issue on my side, so could you please open an issue with more details so we can investigate it further?

@wxiaoguang
Copy link
Contributor

Don't you see this? OIDC users don't stay logged in #28394

zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 22, 2025
* giteaofficial/main:
  Fix various permission & login related bugs (go-gitea#36002)
  Allow empty commit when merging pull request with squash style (go-gitea#35989)
  [skip ci] Updated translations via Crowdin
  Mention proc-receive in text for dashboard.resync_all_hooks func (go-gitea#35991)
  Update JS deps (go-gitea#35978)
  wiki: reuse selectable style for wiki (go-gitea#35990)
@strk
Copy link
Member Author

strk commented Dec 16, 2025

Last I checked, gitea.com was not performing as well as github.com at least for european users, it often had multi-second page load times, so I refrained from using it.

Wouldn't this be an opportunity to improve performance in Gitea ?

Also remember the barrier of entry for new users is much lower on github.com where everyone already has an account.

If I recall correctly Gitea does support login-via-github

@strk strk deleted the reword-sync-repository-hooks-text branch December 16, 2025 09:42
@silverwind
Copy link
Member

silverwind commented Dec 16, 2025

Could you evaluate it again now? I believe the performance should be fast enough at this point.

It is certainly better than before, but it's still not quite the level as snappiness I'd expect. Here comparing repo home page:

  • gitea.com takes 650ms to deliver the HTML response with 200ms TCP latency
  • github.com takes 85ms to deliver the HTML response with 32ms TCP latency
  • my local gitea instance takes 26ms to deliver the HTML response with 0ms TCP latency

So gitea.com takes ~350ms just to render the HTML body. The bigger a gitea instance gets the slower it gets to render body, which should not be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-update-needed The document needs to be updated synchronously lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants