Skip to content

Adding gitea provider #226

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.

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

Merged
merged 9 commits into from
Jun 6, 2023
Merged

Adding gitea provider #226

merged 9 commits into from
Jun 6, 2023

Conversation

souleb
Copy link
Member

@souleb souleb commented Jun 5, 2023

Description

fixes #147

This PR completes #148 by correcting and adding missing features and tests cases.

Note: Some features are not yet available

  • only single file can be committed (support for multiple files is coming)
  • branch can only be created from another branch (branch creation from arbitrary commit is coming)
  • editing teams permission is not supported (because it would edit the given permission for all repositories the team has access to).

malarinv and others added 3 commits May 30, 2023 15:17
Signed-off-by: Malar Kannan <[email protected]>
2. underscored unused args
3. commented exported api
4. enabled custom domain auth_test cases
5. typo, unuse var fixes
6. pagination support and bugfixes

Signed-off-by: Malar Kannan <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jun 5, 2023

Codecov Report

Patch coverage: 64.08% and project coverage change: +0.20 🎉

Comparison is base (687f4f9) 59.48% compared to head (7ba58e0) 59.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #226      +/-   ##
==========================================
+ Coverage   59.48%   59.69%   +0.20%     
==========================================
  Files          44       94      +50     
  Lines        3019     7473    +4454     
==========================================
+ Hits         1796     4461    +2665     
- Misses       1038     2383    +1345     
- Partials      185      629     +444     
Impacted Files Coverage Δ
gitea/client_repository_tree.go 0.00% <0.00%> (ø)
github/client_repository_file.go 58.33% <ø> (-1.13%) ⬇️
gitea/resource_commit.go 27.77% <27.77%> (ø)
gitea/resource_teamaccess.go 32.20% <32.20%> (ø)
gitea/client_repository_teamaccess.go 37.66% <37.66%> (ø)
gitea/client_repository_file.go 48.14% <48.14%> (ø)
gitea/client_repository_deploykey.go 52.47% <52.47%> (ø)
gitea/client_repository_pullrequest.go 56.60% <56.60%> (ø)
gitea/client_organizations.go 60.71% <60.71%> (ø)
gitea/util.go 61.90% <61.90%> (ø)
... and 10 more

... and 31 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Signed-off-by: Soule BA <[email protected]>
@souleb
Copy link
Member Author

souleb commented Jun 5, 2023

@malarinv this is based on your work. We should have a single PR. Either give me access to your branch so I can push those changes, or close your existing pr.

@souleb souleb requested a review from makkes June 6, 2023 09:51
@@ -33,7 +33,7 @@ type BranchClient struct {
}

// Create creates a branch with the given specifications.
// Creating a branch from a commit is noy supported by Gitea.
// Creating a branch from a commit is noy supported by Gitea, the sha refers to the branch to create from.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// Creating a branch from a commit is noy supported by Gitea, the sha refers to the branch to create from.
// Creating a branch from a commit is not supported by Gitea, the `sha` refers to the branch to create from.

Copy link
Member

@makkes makkes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - great job!

@souleb souleb merged commit 251cafb into main Jun 6, 2023
@souleb souleb deleted the soule-gitea branch June 6, 2023 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gitea Support
4 participants