Commit Graph

136 Commits (try-new-ui)

Author SHA1 Message Date
Johannes Batzill bcc9fc50f1 fix: [CODE-3338]: remove pr author from list of reviewers in activities (#3536) 2025-03-12 16:26:28 +00:00
Marko Gaćeša f365ba6713 fix: [CODE-3272]: revert delete of source branch in merge git call (#3542)
* revert delete of source branch in merge git call
2025-03-12 14:48:32 +00:00
Darko Draskovic bb82438965 feat: [CODE-3278]: Add PR activity for reviewers that were added on PR creation (#3499)
* Reuse PullRequestActivityPayloadReviewerAdd and amend it with PrinciaplIDs
* Remove unused and redundant activity (sub)type
* Merge remote-tracking branch 'origin/main' into dd/pr-reviewer-creation-activity
* Replace store individual activity per reviewer added with store batch
* Add PR activity for reviewers that were added on PR creation
2025-03-03 16:07:01 +00:00
Darko Draskovic cad0fbdf98 feat: [CODE-3256]: Add merge verify rule info related to minimum required approvals count for default reviewers (#3483)
* Merge remote-tracking branch 'origin/main' into dd/merge-verify-rules
* Replace princiapl ids with infos in merge verify response
* Populate DefaultReviewerApprovalsResponse with principal ids and current count
* Add MergeVerifyRulesResponse type
2025-02-26 17:56:59 +00:00
Darko Draskovic df7ac77361 feat: [CODE-3233]: Automatically add default reviewers based on rules (#3464)
* Fix tests
* Amend on protection rules verify to include the entire rule output data
* Merge remote-tracking branch 'origin/main' into dd/rule-reviewers
* Fix protection.Branch init in mapToBranchRules
* Amend MergeVerify to include number of default reviewer approvals
* Merge remote-tracking branch 'origin/main' into dd/rule-reviewers
* Refactor add code owners as reviewers to include add default reviewers
* Automatically add default reviewers based on rules
2025-02-25 15:52:59 +00:00
Marko Gaćeša 53142b18a8 techdebt: [CODE-3218]: use context.WithoutCancel in contextutil (#3457) 2025-02-20 19:20:48 +00:00
Darko Draskovic e39ae83e78 feat: [CODE-2987]: Add codeowners as reviewers (#3439)
* Handle violations in branch CreatePullReqVerify
* Merge remote-tracking branch 'origin/main' into dd/codeowner-reviewers
* Add PullReq infix to Create rule interface and dedup reviewers in PR create
* Add create field to pullreq field in rule def
* Add codeowners as reviewers
2025-02-20 16:03:06 +00:00
Darko Draskovic 13f6558ca3 techdebt: [CODE-3169]: Move unnecessary DB calls related to reviewer creation out of the create PR tx (#3420)
* Move activity creation out of tx and reset pr act seq after pr creation
* Add storeCreateReviewerActivity
* Move unnecessary DB calls related to reviewer creation out of the create PR tx
2025-02-13 11:14:36 +00:00
Marko Gaćeša a906259562 feat: [CODE-3148]: refactor refcache (#3390)
* linter issues fix
* reintroduce repofinder to aiagent
* Merge remote-tracking branch 'origin/main' into mg/cache/space-and-repo-finder
* removed refcache deps from aiagent
* pr comments
* fix linter issues
* refactor refcache
2025-02-12 12:42:12 +00:00
Darko Draskovic 72e7a3dde9 techdebt: [CODE-3169]: Move unnecessary DB calls related to label assignment out of the create PR tx (#3402)
* Merge remote-tracking branch 'origin/main' into dd/remove-from-transaction
* Move storeLabelAssignActivity to tx
* Move pr backfill with label info and activity storing out of the tx
* Replace slices containing label/value info with an assignment map
* Move unnecessary DB calls related to label assignment out of the create PR tx
2025-02-11 18:44:17 +00:00
Marko Gaćeša 530a707052 feat: [CODE-3076]: update several refs on merge (#3355)
* resolve pr comments
* update several refs on merge
2025-02-03 10:52:01 +00:00
Enver Biševac 92e39dee79 feat: [CODE-3129]: add ignore whitespace for diff (#3327)
* better parsing of query param ignore_whitespace
* replace hideWhitespace with ignoreWhitespace
* replace rest client literals with vars
* added openapi
* add ignore whitespace for diff
2025-01-31 21:48:57 +00:00
Marko Gaćeša 2fa3914fa9 feat: [CODE-3076]: create and update PR refs in the PR controller (#3319)
* update pr refs in controller
2025-01-27 13:38:55 +00:00
Marko Gaćeša f016ac9ac9 feat: [CODE-3030]: introduce include_git_stats param for list PRs APIs (#3284)
* introduce include_git_stats param for list PRs APIs
2025-01-17 11:27:23 +00:00
Darko Draskovic 6554c472d1 feat: [PIPE-24548]: Add label creation to pullreq creation (#3276)
* Refactor label select base const and its use
* Add created labels to create pr response
* Merge remote-tracking branch 'origin/main' into dd/pr-creation-label-assign
* Add label creation to pullreq creation
2025-01-16 16:48:56 +00:00
Atefeh Mohseni Ejiyeh f78f767ae2 feat: [CODE-2818]: Add Archive Repo functionality (#3051) 2025-01-13 19:02:51 +00:00
Marko Gaćeša 2b959817d5 feat: [CODE-2862]: put space fetch behind a cache (#3201)
* replaced type of the inner cache to SpaceCache
* put space fetch behind a cache
2024-12-25 11:37:40 +00:00
Darko Draskovic 9d4fc2c796 feat: [CODE-2975]: Add reviewers creation to pull req creation (#3182)
* Put pull req creation in tx
* Fix auth error handling and struct/json key/field name
* Merge remote-tracking branch 'origin/main' into dd/pr-create-reviewers
* Move create reviewers functionality into a helper func
* Add reviewers creation to pull req creation
2024-12-23 14:48:52 +00:00
Darko Draskovic 265eb40f22 feat: [CODE-1353]: Add status check and merge check SSE events (#3146)
* Merge remote-tracking branch 'origin/main' into dd/sse-events
* Add webhook create/update/delete event
* Add branch/tag created/updated deleted events
* Merge remote-tracking branch 'origin/main' into dd/sse-events
* Merge remote-tracking branch 'origin/main' into dd/sse-events
* Merge remote-tracking branch 'origin/main' into dd/sse-events
* Add new event types
* Merge remote-tracking branch 'origin/main' into dd/sse-events
* Add logs directly to publisher
* Merge remote-tracking branch 'origin/main' into dd/sse-events
* Add status check and merge check SSE events
2024-12-18 16:00:58 +00:00
Akhilesh Pandey 694e790091 feat: [CODE-2679]: Comment status update changes (#2943) (#3006)
* feat: [CODE-2472]: update wire
* feat: [CODE-2472]: update wire
* feat: [CODE-2472]: update wire
* feat: [CODE-2742]: update comments
* Merge branch 'main' into akp/CODE-2472
* feat: [CODE-2472]: naming and comment
* Apply suggestion from code review
* feat: [CODE-2472]:  remove file
* feat: [CODE-2472]:  Created issue
* feat: [CODE-2472]:  Review
* feat: [CODE-2472]:  rebase
* feat: [CODE-2679]: Comment status update changes (#2943)

* rebase
* feat: [CODE-2679]: review
* feat: [CODE-2679]: wire changes
* feat: [CODE-2679]: Comment status update changes
* feat: [CODE-2679]: Comment status update changes
* feat: [CODE-2472]: fix minor issues
* feat: [CODE-2472]: Update the timestamps
* feat: [CODE-2472]: Pullreq comment updated changes
* feat: [CODE-2472]: changes
* feat: [CODE-2472]: update name to edited
* feat: [CODE-2472]: update name to edited
* feat: [CODE-2472]: status change comment changes
* feat: [CODE-2472]: Text changes only
* feat: [CODE-2472]: initial code
2024-12-18 06:27:39 +00:00
Darko Draskovic c05e7057dc feat: [CODE-1353]: Add SSE events for rules and reviewer add/remove (#3141)
* Add SSE events
2024-12-10 11:24:31 +00:00
Akhilesh Pandey ab72ca9185 feat: [CODE-2472]: CODE Comment updated webhook (#2850)
* feat: [CODE-2472]: review
* feat: [CODE-2742]: review changes
* feat: [CODE-2742]: cast extractor code return
* feat: [CODE-2742]: wire changes
* feat: [CODE-2742]: wire changes
* Merge branch 'main' into akp/CODE-2472
* Apply suggestion from code review
* review changes
* Apply suggestions from code review
* feat: [CODE-2472]: remove commit info
* feat: [CODE-2472]: remove json tags
* feat: [CODE-2472]: wire changes
* feat: [CODE-2472]: fix minor issues
* Merge branch 'main' into akp/CODE-2472
* feat: [CODE-2472]: Update the timestamps
* feat: [CODE-2472]: Pullreq comment updated changes
* feat: [CODE-2472]: changes
* feat: [CODE-2472]: update name to edited
* feat: [CODE-2472]: update name to edited
* Merge branch 'main' into akp/CODE-2472
* feat: [CODE-2472]: status change comment changes
* feat: [CODE-2472]: Text changes only
* feat: [CODE-2472]: initial code
2024-11-14 22:48:04 +00:00
Marko Gaćeša 0db33abeb1 feat: [PIPE-22454]: add PR metadata in get/list PR API response (#2912)
* add PR metadata in PR API response
2024-11-05 12:02:44 +00:00
Akhilesh Pandey b848c642fc feat: [CODE-2425]: Add webhook and trigger for all comment types (#2815)
* feat: [CODE-2425]: Arg format
* Merge branch 'main' into akp/CODE-2425
* feat: [CODE-2425]: return immediately in notification service if its a code-comment
* feat: [CODE-2425]: send gitness comment type
* Merge branch 'main' into akp/CODE-2425
* feat: [CODE-2425]: Add webhook and trigger for all comment types
2024-10-30 16:32:06 +00:00
Akhilesh Pandey b5955c2da9 feat: [CODE-2523]: Audit log bypassed branch deletions (#2851)
* feat: [CODE-2523]: Audit log bypassed branch deletions
2024-10-26 18:47:44 +00:00
Marko Gaćeša 35b7745b65 fix: [CODE-2583]: fix merge dry_run with no method; add dry_run_rules to merge (#2862)
* default to "merge" method for dry-run of merge
* fix merge dry_run with no method; add dry_run_rules to merge
2024-10-25 11:16:37 +00:00
Marko Gaćeša 5eb79b3805 feat: [PIPE-22290]: add additional info to list branches API (#2831)
* change type of branch.SHA from string to sha.SHA
* remove code duplication, changed DB query for PRs
* add additional info to list branches API
2024-10-24 13:19:35 +00:00
Marko Gaćeša 8c5e7546c3 feat: [CODE-2567]: improve git commit message handling (#2849)
* make the change backward compatible
* improve git commit message handling
2024-10-24 10:08:36 +00:00
Johannes Batzill 7b5490ac10 fix: [CODE-2583]: fix dry-running merge without merge method (#2852)
* fix: [CODE-2583]: fix dry-running merge without merge method
2024-10-22 23:06:12 +00:00
Karan Saraswat 9a12d72a10 feat: [PIPE-22618]: API to fetch PR with the source and target branch (#2838)
* fixed go lint error
* fixed swagger endpoint
* feat: [PIPE-22618]: API to fetch PRs with the source and target branch
2024-10-22 11:45:10 +00:00
Akhilesh Pandey c282082cde feat: [CODE-2366]: new webhook event for label assignment (#2810)
* feat: [CODE-2336]: lint
* Merge branch 'main' into akp/CODE-2336
* feat: [CODE-2336]: nit: lint
* feat: [CODE-2336]: nit: rename
* update event and webhook payload
* fix: [CODE-2366]: undo css
* feat: [CODE-2336]: remove service usage
* feat: [CODE-2336]: remove service usage
* Merge remote-tracking branch 'origin/akp/CODE-2336' into akp/CODE-2336

# Conflicts:
#	app/services/webhook/handler_pullreq.go
* feat: [CODE-2336]: remove service usage
* Apply suggestion from code review
* feat: [CODE-2336]: update code to fetch label with values from service
* feat: [CODE-2366]: lint
* feat: [CODE-2366]: new webhook event for label assignment
2024-10-21 15:13:33 +00:00
Marko Gaćeša 069a35a4d8 feat: [CODE-2484]: add API to squash commits on a branch (#2793)
* add openapi entry for squash
* minor fixes
* squash api
2024-10-15 16:49:02 +00:00
Marko Gaćeša 4d577bb378 fix: [CODE-2134]: replace include_description with exclude_description param (#2808) 2024-10-14 18:08:07 +00:00
Akhilesh Pandey 3ef7f9f55a feat: [CODE-2312]: Update UI labels for audit service (#2805)
* feat: [CODE-2312]: revert go-sdk changes in all places
* feat: [CODE-2312]: revert go.mod changes
* feat: [CODE-2312]: fresh reset
* feat: [CODE-2312]: go.mod changes
* feat: [CODE-2312]: address resourcename to commit.go
* Merge branch 'main' into akp/CODE-2312
* feat: [CODE-2312]: address resourcename to commit.go
* feat: [CODE-2312]: address resourcename to commit.go
* feat: [CODE-2312]: address failure due to exceeding limit on resource labels
* feat: [CODE-2312]: lint
* feat: [CODE-2312]: MERGE
* feat: [CODE-2312]: Update wire
* feat: [CODE-2312]: update ui labels
* feat: [CODE-2312]: update resource and action
* feat: [CODE-2312]: Audit log new action : bypassed
* feat: [CODE-2312]: audit log object update
* Merge branch 'main' into akp/CODE-2312
* feat: [CODE-2312]: audit logging changes for commit, merge, create and delete branch
* Merge branch 'main' into akp/CODE-2312
* Merge branch 'main' into akp/CODE-2312
* feat: [CODE-2312]: introduce parser and enclosing method
* feat: [CODE-2312]: annotate error
* feat: [CODE-2312]: name change and todo removal
* feat: [CODE-2312]: Annotate errors
* feat: [CODE-2232]: Branch Rules: UserGroup support: Create and List
2024-10-11 17:05:19 +00:00
Akhilesh Pandey e3b3e983fd feat: [CODE-2312]: audit logging changes for commit, merge, create and delete branch (#2686)
* feat: [CODE-2312]: address resourcename to commit.go
* feat: [CODE-2312]: address failure due to exceeding limit on resource labels
* feat: [CODE-2312]: lint
* feat: [CODE-2312]: MERGE
* feat: [CODE-2312]: Update wire
* feat: [CODE-2312]: update ui labels
* feat: [CODE-2312]: update resource and action
* feat: [CODE-2312]: Audit log new action : bypassed
* feat: [CODE-2312]: audit log object update
* Merge branch 'main' into akp/CODE-2312
* feat: [CODE-2312]: audit logging changes for commit, merge, create and delete branch
* Merge branch 'main' into akp/CODE-2312
* Merge branch 'main' into akp/CODE-2312
* feat: [CODE-2312]: introduce parser and enclosing method
* feat: [CODE-2312]: annotate error
* feat: [CODE-2312]: name change and todo removal
* feat: [CODE-2312]: Annotate errors
* feat: [CODE-2232]: Branch Rules: UserGroup support: Create and List
2024-10-08 16:20:56 +00:00
Marko Gaćeša 2892378036 feat: [CODE-1474]: add check PR rebaseability (#2763)
* update migration number
* rename UpdateConflict->UpdateMergeOutcome
* small fixes
* handle mergeability for all merge methods
* alway separate rebaseability check
* add check PR rebaseability
2024-10-04 12:36:32 +00:00
Marko Gaćeša 8dac8c2797 feat: [CODE-2335]: add dryrun option for rebase API (#2721)
* dryrun rebase
2024-09-23 14:46:17 +00:00
Marko Gaćeša 7c83c5520e feat: [CODE-2385]: add fast-forward merge (#2718)
* add fast-forward merge
2024-09-23 11:34:19 +00:00
Karan Saraswat 8bc9714c92 feat: [CODE-2370]: changes for using PR source branch delete/restore API (#2702)
* fixed error for branch not found
* swagger.yaml prettier
* removed commitSha queryParam and do a getBranch to get the commitSha
* fixed lint issue
* Merge branch 'main' of https://git0.harness.io/l7B_kbSEQD2wjrM7PShm5w/PROD/Harness_Commons/gitness into CODE-2370-2
* added commit_sha query param to PR delete branch API
* fixed linting issues
* updated swagger
* feat: [CODE-2370]: changes for using PR source branch delete/restore API
2024-09-20 08:13:50 +00:00
Karan Saraswat 93b642b2ba feat: [CODE-2359]: PR source branch delete/restore API (#2690)
* passed 409 for open PRs for restore branch API
* addressed review comments
* fixed lint issue
* feat: [CODE-2359]: PR source branch delete/restore API
2024-09-18 12:17:21 +00:00
Marko Gaćeša cafa0148d8 feat: [CODE-2324]: added reviewer-add PR activity entry (#2698)
* added reviewer-add PR activity entry
2024-09-17 15:45:01 +00:00
Marko Gaćeša 1e644163d3 deprecate PR edited field (#2692)
* deprecate PR edited field
2024-09-16 15:44:17 +00:00
Akhilesh Pandey a2cea52155 feat: [CODE-2232]: Branch Rules: UserGroup support: Create and List (#2640)
* feat: [CODE-2327]: add usergroup change to rebase api
* Merge branch 'main' into akp/CODE-2327
* feat: [CODE-2327]: wrap error
* feat: [CODE-2327]: move resolver inside bypass.go and update tests
* feat: [CODE-2327]: newline
* feat: [CODE-2327]: update wire
* Merge remote-tracking branch 'origin' into akp/CODE-2327
* feat: [CODE-2327]: update tests
* feat: [CODE-2327]: fix build
* feat: [CODE-2327]: merge main
* feat: [CODE-2327]: add and update unit tests
* feat: [CODE-2327]: fix
* feat: [CODE-2312]: introduce parser and enclosing method
* Apply suggestion from code review
* feat: [CODE-2327]: annotate error
* Merge branch 'main' into akp/CODE-2327
* feat: [CODE-2327]: usergroup resolver via controller
* feat: [CODE-2312]: update interface in protection
* feat: [CODE-2312]: export and update deduplication use
* feat: [CODE-2327]: Branch Rules: Allow Group bypass.go
* feat: [CODE-2232]: Branch Rules: UserGroup support: Create and List
2024-09-11 08:04:31 +00:00
Karan Saraswat 644678f50c feat: [CODE-2339]: Send sha in delete branch API and fix dry run rules behaviour (#2672)
* go lint issues and added return in the Deletebranch method
* removed the comment
* fixed go lint issues
* fixed query param in swagger
* Merge branch 'main' of https://git0.harness.io/l7B_kbSEQD2wjrM7PShm5w/PROD/Harness_Commons/gitness into ks/CODE-2339
* feat: [CODE-2339]: Send sha in delete branch API and fix dry run rules behaviour
2024-09-09 11:45:18 +00:00
Johannes Batzill 77b01718bd feat: [CODE-2179]: Add error message to protection rule violation error responses (#2665) 2024-09-06 18:54:15 +00:00
Marko Gaćeša ccf267f1cc update pull request edited field after successful merge (#2625)
* update pull request edited field after successful merge
2024-09-02 08:59:50 +00:00
Marko Gaćeša 75042e9293 add pullreq list API for spaces (#2606)
* moved the service to the existing pullreq package
* add pullreq list API for spaces
2024-08-29 11:49:33 +00:00
Marko Gaćeša 6b12b45950 add log entries for rejected merges (#2604)
* add log entries for rejected merges
2024-08-28 12:51:42 +00:00
Karan Saraswat 55d2b1dee3 feat: [CODE-2297]: add dry run param for delete branch API (#2590)
* Merge branch 'main' of https://git0.harness.io/l7B_kbSEQD2wjrM7PShm5w/PROD/Harness_Commons/gitness into CODE-2297
* updated comment
* worked on review comments and added to openapi spec
* return empty violations array when dry_run = true
* feat: [CODE-2297]: add dry run param for delete branch API
2024-08-28 08:39:04 +00:00
Akhilesh Pandey 0578be8afe feat: [CODE-1941]: user group changes (#2134)
* feat: [CODE-1941]: remove usergroup filter from request
* feat: [CODE-1941]: decrement page for harness-core
* feat: [CODE-1941]: update pagination request
* feat: [CODE-1941]: change interface from find to list
* feat: [CODE-1941]: change reviewer add permissions
* feat: [CODE-1941]: comments
* feat: [CODE-1941]: comments
* fix wire issues
* feat: [CODE-1941]: remove users from the db table usergroups
* feat: [CODE-1941]: annotate errors
* feat: [CODE-1941]: annotate errors
* feat: [CODE-1941]: sql style
* feat: [CODE-1941]: revert common.go change for parse limit
* feat: [CODE-1941]: move usergroup listing to space level
* feat: [CODE-1941]: update usergroup listing and update delete to push permission
* feat: [CODE-1941]: Update combined listing
* feat: [CODE-1941]: Update combined listing
* feat: [CODE-1941]: format
* feat: [CODE-1941]: rebase
* feat: [CODE-1941]: separate db struct for usergroup.go
* feat: [CODE-1941]: change usergroup filter to default query and review comments
* feat: [CODE-1941]: change usergroup filter to default
* feat: [CODE-1941]: change return type for search service interface
* feat: [CODE-1941]: change return type for search service interface
* feat: [CODE-1941]: change return type for search service interface
* feat: [CODE-1941]: Search Service interface and wire changes
* feat: [CODE-1941]: Search Service interface and wire changes
* feat: [CODE-1941]: Search UserGroups Handler change - codeformat
* feat: [CODE-1941]: Search UserGroups Handler change - codeformat
* feat: [CODE-1941]: Search UserGroups Handler change - codeformat
* feat: [CODE-1941]: Search UserGroups Handler change
* feat: [CODE-1941]: Big change: New LOGIC for UserGroup Reviewers
* feat: [CODE-1941]: DB changes
* feat: [CODE-1941]: listing api
* feat: [CODE-1941]: mapping methods
* feat: [CODE-1941]: list method for pullreq_reviewers db
* feat: [CODE-1941]: database queries
* feat: [CODE-1941]: database queries
* feat: [CODE-2058]: dbstore changes
* feat: [CODE-1941]: Controller and Database code
* feat: [CODE-1941]: update reviewer_add.go code
* feat: [CODE-1941]: update reviewers and usergroup db migrations
* feat: [CODE-1941]: Update API ops
* feat: [CODE-1941]: Update DB ops for reviewers
* feat: [CODE-1941]: db migration update
* feat: [CODE-1941]: db migration update
* feat: [CODE-1941]: code comments
* feat: [CODE-1941]: update migration numbering
* feat: [CODE-1941]: Add postgresql migrations
* feat: [CODE-1941]: Update DB ops for reviewers

feat: [CODE-1941]: Update API ops

feat: [CODE-1941]: update reviewers and usergroup db migrations

feat: [CODE-1941]: update reviewer_add.go code

feat: [CODE-1941]: break pr

feat: [CODE-1941]: break pr

feat: [CODE-1941]: Update migration numbering
* feat: [CODE-1941]: create table usergroups
* feat: [CODE-1941]: table migrations
* feat: [CODE-1941]: user group changes
2024-08-26 15:35:27 +00:00