Skip to content
This repository was archived by the owner on Jun 13, 2025. It is now read-only.

[feat] Enable pagination for AI enabled repos #1101

Merged
rohitvinnakota-codecov merged 23 commits intomainfrom
rvinnakota/update-ai-repo
Feb 24, 2025
Merged

[feat] Enable pagination for AI enabled repos #1101
rohitvinnakota-codecov merged 23 commits intomainfrom
rvinnakota/update-ai-repo

Conversation

@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor

@rohitvinnakota-codecov rohitvinnakota-codecov commented Jan 15, 2025

Updates the current repositories resolver that supports pagination for Codecov AI enabled repos. The UI will be updated to use this updated resolver(and support pagination), after which we can safely delete the old non-paginated resolver.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 15, 2025

Codecov Report

Attention: Patch coverage is 90.90909% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.91%. Comparing base (02ff374) to head (a7be526).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
graphql_api/actions/repository.py 90.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1101      +/-   ##
==========================================
- Coverage   95.92%   95.91%   -0.01%     
==========================================
  Files         492      492              
  Lines       16847    16865      +18     
==========================================
+ Hits        16160    16176      +16     
- Misses        687      689       +2     
Flag Coverage Δ
unit 95.91% <90.90%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@github-actions
Copy link
Copy Markdown
Contributor

✅ All tests successful. No failed tests were found.

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov

@codecov-notifications
Copy link
Copy Markdown

codecov-notifications bot commented Jan 15, 2025

Codecov Report

Attention: Patch coverage is 90.90909% with 2 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
graphql_api/actions/repository.py 90.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review2

@seer-by-sentry
Copy link
Copy Markdown
Contributor

On it! We are reviewing the PR and will provide feedback shortly.

@codecov codecov deleted a comment from seer-by-sentry bot Jan 30, 2025
@codecov codecov deleted a comment from seer-by-sentry bot Jan 30, 2025
@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review2

@seer-by-sentry
Copy link
Copy Markdown
Contributor

On it! We are reviewing the PR and will provide feedback shortly.

@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review2

@seer-by-sentry
Copy link
Copy Markdown
Contributor

seer-by-sentry bot commented Feb 3, 2025

On it! We are reviewing the PR and will provide feedback shortly.

@seer-by-sentry
Copy link
Copy Markdown
Contributor

seer-by-sentry bot commented Feb 3, 2025

No changes requiring review at this time.

1 similar comment
@seer-by-sentry
Copy link
Copy Markdown
Contributor

seer-by-sentry bot commented Feb 3, 2025

No changes requiring review at this time.

@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review2

@codecov codecov deleted a comment from seer-by-sentry bot Feb 3, 2025
@codecov codecov deleted a comment from seer-by-sentry bot Feb 3, 2025
@seer-by-sentry
Copy link
Copy Markdown
Contributor

seer-by-sentry bot commented Feb 3, 2025

Sentry has generated a new PR with unit tests for this PR. View the new PR(#1126) to review the changes.

@seer-by-sentry seer-by-sentry bot mentioned this pull request Feb 3, 2025
@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review2

@codecov codecov deleted a comment from seer-by-sentry bot Feb 3, 2025
@seer-by-sentry
Copy link
Copy Markdown
Contributor

seer-by-sentry bot commented Feb 3, 2025

No changes requiring review at this time.

@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review2

@codecov-ai
Copy link
Copy Markdown
Contributor

codecov-ai bot commented Feb 4, 2025

On it! We are reviewing the PR and will provide feedback shortly.

Copy link
Copy Markdown
Contributor

@ajay-sentry ajay-sentry left a comment

Choose a reason for hiding this comment

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

lgtm thanks for fixing those weird tests

@rohitvinnakota-codecov rohitvinnakota-codecov added this pull request to the merge queue Feb 24, 2025
@rohitvinnakota-codecov rohitvinnakota-codecov removed this pull request from the merge queue due to a manual request Feb 24, 2025
@rohitvinnakota-codecov
Copy link
Copy Markdown
Contributor Author

@codecov-ai-reviewer review

@codecov-ai
Copy link
Copy Markdown
Contributor

codecov-ai bot commented Feb 24, 2025

On it! We are reviewing the PR and will provide feedback shortly.

@codecov-ai
Copy link
Copy Markdown
Contributor

codecov-ai bot commented Feb 24, 2025

No changes requiring review at this time.

@rohitvinnakota-codecov rohitvinnakota-codecov added this pull request to the merge queue Feb 24, 2025
@rohitvinnakota-codecov rohitvinnakota-codecov removed this pull request from the merge queue due to a manual request Feb 24, 2025
@rohitvinnakota-codecov rohitvinnakota-codecov added this pull request to the merge queue Feb 24, 2025
Merged via the queue into main with commit baeff88 Feb 24, 2025
24 of 28 checks passed
@rohitvinnakota-codecov rohitvinnakota-codecov deleted the rvinnakota/update-ai-repo branch February 24, 2025 16:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants