Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Docs: Update the feature comparison to other Git Hosting Services #20933

Merged
merged 2 commits into from
Oct 17, 2022
Merged

Docs: Update the feature comparison to other Git Hosting Services #20933

merged 2 commits into from
Oct 17, 2022

Conversation

delvh
Copy link
Member

@delvh delvh commented Aug 23, 2022

This was drastically outdated recently.

@delvh
Copy link
Member Author

delvh commented Aug 23, 2022 •
edited
Loading

Review hints:

  1. I reordered the general features by feature relevance
  2. I added most major missing features. From around Gitea 1.14 on, however, I lost concentration and still wanted to finish it today. So, it's possible I missed some major features since then. If that's the case, feel free to add them.
  3. Due to the sheer amount of cross- and checkmarks, it is possible that something is wrong. Please double-check!
  4. As I'm not capable of Chinese, I only updated the English file, meaning they are now out of sync. Chinese contributors are welcome to add a translation, or to accept that the files now have different content.
  5. I removed all links and other text for hosters that are not Gitea for consistency
  6. For Gitea, I removed all checkmark links and added links to all crossmarks/ slashes (if possible). Again, simply to be consistent.
  7. As I don't have access to any of the other instances, I had to guess if they support it. My approach was to google the feature combined with the hoster and see if something is mentioned in their docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 23, 2022
@delvh delvh added type/docs This PR mainly updates/creates documentation skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Aug 23, 2022
@delvh delvh added this to the 1.18.0 milestone Aug 23, 2022
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 23, 2022
@delvh delvh changed the title Update the feature comparison to other Git Hosting Services Docs: Update the feature comparison to other Git Hosting Services Aug 25, 2022
@delvh delvh mentioned this pull request Aug 27, 2022
Closed
@lunny
Copy link
Member

lunny commented Oct 17, 2022

Please resolve conflicts

@delvh
Copy link
Member Author

delvh commented Oct 17, 2022

Done.

@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 Oct 17, 2022
@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 Oct 17, 2022
@techknowlogick techknowlogick merged commit 6771b10 into go-gitea:main Oct 17, 2022
@delvh delvh deleted the documentation/update-feature-comparison branch October 17, 2022 21:46
zjjhot added a commit to zjjhot/gitea that referenced this pull request Oct 18, 2022
@zjjhot
Merge remote-tracking branch 'upstream/main'
b55a86c 
* upstream/main: (32 commits)
  inline gitpod image (go-gitea#21494)
  [skip ci] Updated translations via Crowdin
  Do not send notifications for draft releases (go-gitea#21451)
  Update reverse-proxies.zh-cn.md (go-gitea#21484)
  Docs: Update the feature comparison to other Git Hosting Services (go-gitea#20933)
  Add some api integration tests (go-gitea#18872)
  probe if sha before exec git (go-gitea#21467)
  Fix incorrect notification commit url (go-gitea#21479)
  Localize all timestamps (go-gitea#21440)
  [skip ci] Updated translations via Crowdin
  Add system setting table with cache and also add cache supports for user setting (go-gitea#18058)
  Return 404 when user is not found on avatar (go-gitea#21476)
  Enforce grouped NuGet search results (go-gitea#21442)
  Display total commit count in hook message (go-gitea#21400)
  Refactor GetNextResourceIndex to make it work properly with transaction (go-gitea#21469)
  Simplify fmt-check (go-gitea#21458)
  update current stable version
  1.17.3 changelog
  [skip ci] Updated translations via Crowdin
  Fix mermaid-related bugs (go-gitea#21431)
  ...
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@techknowlogick techknowlogick techknowlogick approved these changes

@lunny lunny lunny approved these changes

Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/docs This PR mainly updates/creates documentation
Projects
None yet
Milestone
1.18.0
Development

Successfully merging this pull request may close these issues.
4 participants
@delvh @lunny @techknowlogick @GiteaBot