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.

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

[CIVIS-2938] Add Git::SearchCommits api client #152

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

anmarchenko
Copy link
Member

What does this PR do?
Adds Datadog::CI::Git::SearchCommits class that sends backen API request to /api/v2/git/repository/search_commits endpoint and gets a list of commits that are already known to the backend DB.

How to test the change?
For now unit tests only, this class is not used yet

@anmarchenko anmarchenko requested review from a team as code owners April 5, 2024 14:22
@anmarchenko anmarchenko changed the title add Git::SearchCommits callable that sends backend API request to check which commits are already known to the backend [CIVIS-2938] Add Git::SearchCommits api client Apr 5, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.03%. Comparing base (9716194) to head (6a57b34).

Additional details and impacted files
@@            Coverage Diff             @@
##              1.0     #152      +/-   ##
==========================================
+ Coverage   99.02%   99.03%   +0.01%     
==========================================
  Files         179      181       +2     
  Lines        8090     8202     +112     
  Branches      339      344       +5     
==========================================
+ Hits         8011     8123     +112     
  Misses         79       79              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

…ck which commits are already known to the backend
@anmarchenko anmarchenko force-pushed the anmarchenko/search_commits_api branch from 6a57b34 to 07111cd Compare April 5, 2024 15:32
@anmarchenko anmarchenko merged commit e238a74 into 1.0 Apr 5, 2024
23 checks passed
@anmarchenko anmarchenko deleted the anmarchenko/search_commits_api branch April 5, 2024 15:35
@anmarchenko anmarchenko added this to the 1.0 milestone Apr 23, 2024
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.

3 participants