Skip to content

fix/repo-list: print partial data when there are graphql errors#1282

Open
burmudar wants to merge 4 commits intomainfrom
wb/cpl-297-skip-results
Open

fix/repo-list: print partial data when there are graphql errors#1282
burmudar wants to merge 4 commits intomainfrom
wb/cpl-297-skip-results

Conversation

@burmudar
Copy link
Copy Markdown
Contributor

@burmudar burmudar commented Apr 1, 2026

Closes CPL-297

Update src repo list to not fail the whole listing if only some repositories failed.

Changes

  • If we have data and errors in a result, treat the errors as warnings. This allows us to print partial results
  • if we have NO data and errors, treat the errors as fatal errors

Test plan

Unit test - Could only recreate the error case in a unit case since all the repo listing done against s2 + dotcom didn't have errors.

burmudar added 3 commits April 1, 2026 15:51
 - errors are treated as warnings when getting partial data
 - if we have no data and just errors that is a fatal error
@burmudar burmudar self-assigned this Apr 1, 2026
@burmudar burmudar requested a review from a team April 1, 2026 14:36
@burmudar burmudar force-pushed the wb/cpl-297-skip-results branch from ed65bff to 32f7d32 Compare April 1, 2026 14:42
@burmudar burmudar force-pushed the wb/cpl-297-skip-results branch from 32f7d32 to 65b9840 Compare April 1, 2026 14:44
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.

1 participant