Skip to content
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

fix: add ignore to archived and forked repositories on search #56

Merged
merged 1 commit into from
Jul 27, 2023

Conversation

adrielp
Copy link
Collaborator

@adrielp adrielp commented Jul 27, 2023

Many orgs have multiple repos that are forks and archived. adding isArchived: false and isFork: false to the repositories query reduces the overhead of getting metrics on non-maintained repositories and increases performance by reducing the amount of subsequent calls and iterations. This may be feature flagged via config in the future.

Many orgs have multiple repos that are forks and archived. adding
`isArchived: false` and `isFork: false` to the `repositories` query
reduces the overhead of getting metrics on non-maintained repositories
and increases performance by reducing the amount of subsequent calls and
iterations. This may be feature flagged via config in the future.
@adrielp adrielp requested a review from a team as a code owner July 27, 2023 16:02
@github-actions github-actions bot added the go label Jul 27, 2023
Copy link
Contributor

@pmpaulino pmpaulino left a comment

Choose a reason for hiding this comment

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

LGTM

@adrielp adrielp merged commit 52d2552 into main Jul 27, 2023
@adrielp adrielp deleted the ignore branch July 27, 2023 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants