-
Notifications
You must be signed in to change notification settings - Fork 280
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
Avoid fetch only 100 teams #778
Conversation
@microsoft-github-policy-service agree |
@ricardosequeira Some ORGs may have hundreds or thousands of teams. Get all teams at the same time may took a long time and cause performance issue. Can you refactor the top to a custom value with default values like 100? |
@xuzhang3, can you review this PR again? |
@ricardosequeira please add the |
@xuzhang3, done |
Co-authored-by: xuzhang3 <57888764+xuzhang3@users.noreply.github.com>
@ricardosequeira LGTM 🚢 |
All Submissions:
What about the current behavior has changed?
Issue Number: #766
Does this introduce a change to
go.mod
,go.sum
orvendor/
?Does this introduce a breaking change?
Any relevant logs, error output, etc?
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Other information
Closes #766