merge - #2
Open
Yshayy wants to merge 457 commits into
Open
Conversation
Yshayy
marked this pull request as ready for review
May 19, 2021 07:55
(feat) harness, add repo list
(feat) harness, add webhook parsing
fetch branch for bitbucket onprem
(feat) harness, add list commits / branches
(feat) harness, add user and compare branches
(feat) harness, fix create branch, PR calls
(fix) harness, webhook fixes
(feat) harness, add finduser
(maint) prep for 1.29.0
…ndle csrf failures
Add required header for bitbucket server in commit API use-case to handle csrf failures
…and consistency - Renamed methods for fetching repositories and calculating slice indices to improve clarity and maintainability. - Updated method signatures to reflect new naming conventions, ensuring consistency across the codebase. - Enhanced comments to provide better context for method functionality.
…n logic - Renamed test methods to improve clarity and consistency in naming conventions. - Updated test cases to reflect changes in method names related to slice index calculations. - Enhanced comments for better understanding of test functionality and expected outcomes.
feat: [PIPE-32185]: handle deprecated cross-workspace API endpoints
* 3a1cf9 include subgroups for listing gitlab projects
scm/driver/bitbucket: fix multiple dropped errors
… silently swallowing them (#100002) * 10a411 refactor: Clean up error handling and remove unused tests in Bitbucket driver * b1a704 fix: [PIPE-32774]: Removed extra comments * 0d256b fix: [PIPE-32774]: Add UT's * ec9df7 fix: [PIPE-32774]: Surface Bitbucket 429 rate-limit errors instead of silently swallowing them
…point fix: use non-admin endpoint for repository permission checks
… PR webhooks (#100003) * e15e87 fix: revert to time.Unix(ms/1000) — keep call sites unchanged * 32d51a fix: [PIPE-34150]: Use time.UnixMilli, add comments, refresh goldens * dfaef0 fix: [PIPE-34150]: Correct ISO 8601 layout list in epochOrISO unmarshaler * 906822 chore: remove raw payload reference files from testdata * 87c270 test: [PIPE-34150]: Add real-world FNB Bitbucket DC 10.3 push update payload test * 3ac201 test: [PIPE-34150]: Add real-world FNB Bitbucket DC 10.3 push payload test * e5a0b4 fix: [PIPE-34150]: Handle ISO 8601 date strings in Bitbucket DC stash PR webhooks
added url split strategies to be used by gitlab
* 9ec553 Merge branch 'master' of https://git0.harness.io/l7B_kbSEQD2wjrM7PShm5w/PROD/Harness_Commons/go-scm into feat/STO-11679/pr-comment-api-update * 4c15b9 feat:[STO-11679]: cleanup * bb75ba feat:[STO-11679]:Added create comment API changes * d87d12 feat:[STO-11679]:Added create comment API changes
…tus (#358) * feat: [CODE-5421]: add normalized CheckHook + GitHub check_run/status parsing Introduce scm.CheckHook, a provider-agnostic CI-check webhook carrying a batch of scm.Check entries (name, status, conclusion, target URL, sha, timestamps, optional PR). The batch shape accommodates providers that report multiple checks per event (e.g. GitLab pipeline jobs) while GitHub/Bitbucket populate a single entry. GitHub driver now parses check_run and status events (previously ErrUnknownEvent) into CheckHook. Status carries the normalized ExecutionStatus while Conclusion preserves the raw provider outcome so richer GitHub conclusions (neutral, timed_out, ...) are not lost. Adds HookEvents.Check for subscription and fixture/golden tests for both events. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * feat: [CODE-5421]: subscribe GitHub check_run/status via HookEvents.Check Wire the new HookEvents.Check flag through convertFromHookEvents so CreateHook registers both check_run and status GitHub events when a caller subscribes to checks. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
…ntents API (#100006) * 873441 fix: [CDS-125782]: Use int64 for Size field in parity with proto3 * 7c5295 test: [CDS-125782]: Add tests for large file and empty file content responses * facf81 feat: [CDS-125782]: Propagate size and encoding fields from GitHub Contents API
BitBucket closing the PR
- Add workspace and repo fields to wrapper struct - Add SetWorkspace() function to configure workspace - Use workspace when available to avoid account scope requirement - Fix fetchReposWithPagination to check for configured workspace - Update test files to use new wrapper struct format
This reverts commit 66cedce.
…arness Git providers (#100007) * 07f1c1 feat: [IAC-7349]: Support comment list and delete for BitBucket and Harness Git providers
… prefix-based branch search (#100008) * aaa5f2 added a comment that pagination is not supported * 651bf4 removed log line * 991ded added log line * e92876 added log line * 1dd54a added log line * 7e541f added search term support for listing
* 2c7c1b fix:[PIPE-35486]: Add codeowners for go-scm
* 46f249 fix:[PIPE-35486]: Add path in codeowner file
* f9d7d0 feat:[STO-11846]:Removed Patch * f34cd3 feat:[STO-11846]:Reverted complex patches * ac053a feat:[STO-11864]: add diff parsing/applying unit tests * fc8bd9 feat:[STO-11864]: revered non-required changes * 6e2540 feat:[STO-11864]: Added review link * b7a40d added bug fix * 6085ab added bug fix * 6d7427 new commit * 1ed16e new commit * e85589 added review
* 5f7572 feat:[STO-11846]: Renamed method as per comments * 9f1cd9 feat:[STO-11846]: Renamed method as per comments * 5f47ec feat: [STO-11846]: FindFileDiff owns pagination internally; rename number to prNumber * 01c615 feat:[STO-11846]: Fixed unused code and comments * 0416db feat:[STO-11846]: Updated Patch apis * 143dfa feat:[STO-11846]:Added PR file Diff API
* 155c62 fix:[STO-12110]: Added hunk header * 83ec9b feat:[STO-XXX]:new commit
…0019) * 3059ec feat: [CODE-5904]: implement CreateStatus for the harness driver
…sueService (#100018) * 8bafb4 docs: [CODE-5903]: explain reaction ErrNotSupported stubs for bitbucket and azure * 874e6b docs: [CODE-5903]: note issue-comment vs review-comment reaction split in GitHub driver * 30efc3 fix: [CODE-5903]: guard against nil ReactionInput in AddReaction * 1e449d fix: [CODE-5903]: remove ListReactions (no listable endpoint on Harness Code) * 2b4c3c fix: [CODE-5903]: make DeleteReaction's reaction id a string * 1f0a31 feat: [CODE-5903]: implement comment reactions for harness and gitlab drivers * 4c1e4e feat: [CODE-5903]: add comment reaction support to PullService and IssueService
…rvice (#100020) * bea8e9 test: [CODE-5905]: add gock coverage for gitlab EditComment * 1295de docs: [CODE-5905]: explain bitbucket issue EditComment ErrNotSupported * dd2eb9 feat: [CODE-5905]: add EditComment support to PullService and IssueService
… (#100016) * 84f00d feat: subscribe to github merge_group when creating webhooks * 871688 Merge remote-tracking branch 'origin/master' into feat/merge-queue-webhook-parsing * b37650 feat: add merge queue webhook parsing for GitHub driver * 78e736 feat: add merge queue webhook parsing for Harness driver
…00022) * 425da7 fix: [CI-23182]: populate commit Signature.Date in harness driver
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.