Skip to content

Conversation

richard-leon-130524
Copy link
Contributor

Summary of PR changes

  • Since the GitHub GraphQL API restricts queries to a maximum of 100 node IDs, I’ve added logic to iterate over restDeployments in chunks of 100 and queries the GraphQL API for each chunk.
  • Unit test:
    • given 120 deployments, should return 120 statuses
  • dist/index.js build updated

@richard-leon-130524 richard-leon-130524 requested a review from a team as a code owner September 9, 2025 17:18
Copy link
Contributor

@hpractv hpractv left a comment

Choose a reason for hiding this comment

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

I like the additional tests. It looks good it just needs updates to the README file for version numbers.

hpractv
hpractv previously approved these changes Sep 9, 2025
Copy link
Contributor

@hpractv hpractv left a comment

Choose a reason for hiding this comment

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

👍🏽

Copy link
Contributor

@hpractv hpractv left a comment

Choose a reason for hiding this comment

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

👍🏽

@hpractv hpractv merged commit 0f8ea9e into im-open:main Sep 10, 2025
2 checks passed
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.

2 participants