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

Register code repository file tree api #206

Merged

hawkli-1994 merged 1 commit into main from feat/git-repo-file-tree-rest-api

May 27, 2022

Contributor

hawkli-1994 commented May 16, 2022 •

edited

Loading

Changes

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

spec PR link included
Follows the commit message standard
Meets the contributing guidelines (including
functionality, content, code)
Test cases with documentation and functionality works as expected using current and related github repos (MUST deploy and check)
Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

chengjingtao requested changes

View reviewed changes

apis/meta/v1alpha1/gitcommit_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/gitcommit_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/gitrepofiletree_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/gitrepofiletree_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/gitrepofiletree_types.go Show resolved Hide resolved

client/userinfo.go Outdated Show resolved Hide resolved

client/userinfo_test.go Outdated Show resolved Hide resolved

client/userinfo_test.go Outdated Show resolved Hide resolved

client/userinfo_test.go Outdated Show resolved Hide resolved

plugin/route/gitrepofiletree.go Outdated Show resolved Hide resolved

hawkli-1994 force-pushed the feat/git-repo-file-tree-rest-api branch from 625079e to e30ac82 Compare

May 22, 2022 12:22

chengjingtao requested changes

View reviewed changes

client/decorator/filters.go Outdated Show resolved Hide resolved

client/decorator/filters.go Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

apis/meta/v1alpha1/gitcommit_types.go Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

client/filter_test.go Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

client/filters.go Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

client/userinfo_test.go Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

plugin/route/gitrepofiletree_test.go Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

client/filters.go Outdated Show resolved Hide resolved

danielfbm requested changes

View reviewed changes

Contributor

danielfbm left a comment

很多函数，对象，字短都没有任何注释，其他评论都需要改

alaudabot reviewed

View reviewed changes

parallel/page.go Show resolved Hide resolved

parallel/page_test.go Show resolved Hide resolved

alaudabot reviewed

View reviewed changes

user/userinfo_test.go Outdated Show resolved Hide resolved

user/userinfo_test.go Outdated Show resolved Hide resolved

hawkli-1994 force-pushed the feat/git-repo-file-tree-rest-api branch 3 times, most recently from 0494cde to f3e3513 Compare

May 27, 2022 01:24

alaudabot reviewed

View reviewed changes

apis/meta/v1alpha1/gitrepofiletree_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/gitrepofiletree_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/gitrepofiletree_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/user_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/user_types.go Outdated Show resolved Hide resolved

apis/meta/v1alpha1/user_types.go Outdated Show resolved Hide resolved

parallel/page.go Outdated Show resolved Hide resolved

user/context.go Show resolved Hide resolved

user/filters.go Show resolved Hide resolved

user/userinfo.go Show resolved Hide resolved

alaudabot reviewed

View reviewed changes

user/context.go Outdated Show resolved Hide resolved

user/filters.go Outdated Show resolved Hide resolved

user/userinfo.go Outdated Show resolved Hide resolved

alaudabot reviewed

View reviewed changes

user/context.go Outdated Show resolved Hide resolved

user/filters.go Outdated Show resolved Hide resolved

user/userinfo.go Outdated Show resolved Hide resolved

alaudabot reviewed

View reviewed changes

plugin/client/gitfiletree.go Outdated Show resolved Hide resolved

plugin/client/gitfiletree.go Outdated Show resolved Hide resolved

plugin/client/gitfiletree.go Outdated Show resolved Hide resolved

plugin/client/interface.go Outdated Show resolved Hide resolved

Contributor

alaudabot commented May 27, 2022 •

edited

Loading

Quality Gate

_{updated at Fri May 27 11:13:55 UTC 2022}

Status: OK 🟢

Metric	Rating
🐛 Bugs: 0	🏗 Reliability: A 🟢
🤮 Code smells: 0	🛠 Maintainability: A 🟢
⚠️ Vulnerabilities: 0	🦺 Security: A 🟢
🔥 Security hotspots: 0	🔎 Security Review: A 🟢

Metric	Change set	Target _(main)	After
🧪 Code coverage	64.74% _{Coverage on 156 new lines}	32.70%	33.80% ⬆️ 🟢 _{estimated after merge} _{allowed difference 0}
👥 Duplications	0.00% _{Duplications on 0 new lines}	0.30%	0.30% ↔️ 🟢 _{estimated after merge} _{allowed difference 0}

[206]

chengjingtao previously approved these changes

View reviewed changes

hawkli-1994 dismissed chengjingtao’s stale review via

b532a41

May 27, 2022 06:46

hawkli-1994 force-pushed the feat/git-repo-file-tree-rest-api branch from 86fd8ec to b532a41 Compare

May 27, 2022 06:46

danielfbm reviewed

View reviewed changes

go.mod Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

parallel/page_test.go Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

plugin/client/gitfiletree.go Outdated Show resolved Hide resolved

danielfbm reviewed

View reviewed changes

plugin/route/gitrepofiletree.go Show resolved Hide resolved

danielfbm previously approved these changes

View reviewed changes

hawkli-1994 dismissed danielfbm’s stale review via

c3efec1

May 27, 2022 11:01


          Register code repository file tree api

c68b1d6

add notes

fix tree types

Add the function of parsing user information

CreateGitRepoFile add authorinfo

delete fmt print

add commit author info

add notes, agg GetUserBasicInfoFromReq errors

add userinfo filter

add filter unittest

fix userinfo filter

fix log

fix context key type

add exception request filter, parallel page tool

add userinfo filter

fix notes

add notes

fix jwt unittest

fix unittest TestAboutGitRepositoryFileTreeTable

fix notes

fix notes

fix package notes

fix lint: line length

delete exception filter

format: line lenght

add unittest TestPage checkout

clean imports

split test functions

fix unittest TestPageBigNum

hawkli-1994 force-pushed the feat/git-repo-file-tree-rest-api branch from c3efec1 to c68b1d6 Compare

May 27, 2022 11:05

chengjingtao approved these changes

View reviewed changes

hawkli-1994 merged commit 0ee41f1 into main

hawkli-1994 deleted the feat/git-repo-file-tree-rest-api branch

May 27, 2022 11:15

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

Labels

None yet