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)
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