Skip to content

remove two rest api calls to improve rate limit - #2936

Draft
marcoieni wants to merge 1 commit into
mainfrom
remove-two-rest-api-calls-to-improve-rate-limit
Draft

marcoieni wants to merge 1 commit into
mainfrom
remove-two-rest-api-calls-to-improve-rate-limit

Conversation

@marcoieni

@marcoieni marcoieni commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

AI disclosure

I used Opus 5.5 to generate this change. I reviewed its output and changed it where necessary.

}
}
}
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

the info that those rest API calls were getting are now retrieved in this graphql call.

@marcoieni
marcoieni force-pushed the remove-two-rest-api-calls-to-improve-rate-limit branch 3 times, most recently from ea5755c to 28d6224 Compare November 10, 2026 11:23
@github-actions

github-actions Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Dry-run check results

[WARN  rust_team::sync] sync-team is running in dry mode, no changes will be applied.
[INFO  rust_team::sync] synchronizing crates-io
[INFO  rust_team::sync] synchronizing github
[INFO  rust_team::sync] Rate limits before: REST 15000/15000 remaining, resets at 2026-11-10 16:40:34 UTC, GraphQL 12500/12500 remaining, resets at 2026-11-10 16:40:34 UTC
[INFO  rust_team::sync] Rate limits after: REST 14526/15000 remaining, resets at 2026-11-10 16:40:44 UTC, GraphQL 12411/12500 remaining, resets at 2026-11-10 16:40:39 UTC
[INFO  rust_team::sync] Consumed 474 REST calls and 89 GraphQL calls

@marcoieni
marcoieni force-pushed the remove-two-rest-api-calls-to-improve-rate-limit branch from 28d6224 to 9584944 Compare November 10, 2026 11:30
Comment on lines -736 to -740
// Exclude inherited organization/enterprise rulesets, as we only manage
// repository rulesets.
// Note that parent rulesets can contain parent-only rule types, such as `workflows`,
// so if you enable this, make sure the deserialization of the ruleset can handle all
// organization/enterprise rule types, too.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this comment is now in the graphql call. Line 432

@marcoieni
marcoieni force-pushed the remove-two-rest-api-calls-to-improve-rate-limit branch 4 times, most recently from 987dc3f to cfe973b Compare November 10, 2026 12:55
@marcoieni
marcoieni force-pushed the remove-two-rest-api-calls-to-improve-rate-limit branch from cfe973b to 58a7cbc Compare November 10, 2026 15:34

This branch has not been deployed

No deployments
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