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.

Sign up for GitHub

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

enh: use graphql to consolidate api calls #267

Merged
merged 8 commits into from
Mar 18, 2025
Merged

enh: use graphql to consolidate api calls #267

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
afcca0b
Enh: move to graphql for metrics
lwasser Mar 5, 2025
631b19b
Fix: numerous bugs and cleanup
lwasser Mar 5, 2025
38d9f56
fix: remove metrics object no longer needed & cleanup
lwasser Mar 6, 2025
1818ac7
fix: generalize return_response
lwasser Mar 6, 2025
14d1b6c
tests-rate limiting
lwasser Mar 6, 2025
337daa1
fix: tests for api auth
lwasser Mar 13, 2025
195c6c0
Update src/pyosmeta/github_api.py
lwasser Mar 14, 2025
2aefca7
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Mar 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,3 +157,5 @@ src/pyosmeta/_version.py
*.pickle
.token
token.txt

.bash_profile
6 changes: 2 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
[![Publish to PyPI](https://github.com/pyOpenSci/pyosMeta/actions/workflows/publish-pypi.yml/badge.svg)](https://github.com/pyOpenSci/pyosMeta/actions/workflows/publish-pypi.yml)
[![.github/workflows/test-run-script.yml](https://github.com/pyOpenSci/pyosMeta/actions/workflows/test-run-script.yml/badge.svg)](https://github.com/pyOpenSci/pyosMeta/actions/workflows/test-run-script.yml)


## Description

**pyosmeta** provides the tools and scripts used to manage [pyOpenSci](https://pyopensci.org)'s contributor and peer
Expand All @@ -25,18 +24,17 @@ This repo contains a small module and several CLI scripts, including:

_Since pyOpenSci uses this tool for its website, we expect this package to have infrequent releases._


## Installation

Using pip:

```
```console
pip install pyosmeta
```

Using conda:

```
```console
conda install pyosmeta
```

Expand Down
10 changes: 3 additions & 7 deletions src/pyosmeta/cli/process_reviews.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,6 @@ def main():
process_review = ProcessIssues(github_api)

# Get all issues for approved packages - load as dict
# TODO: this doesn't have to be in process issues at all. it could fully
# Call the github module
issues = process_review.get_issues()
accepted_reviews, errors = process_review.parse_issues(issues)
for url, error in errors.items():
Expand All @@ -48,11 +46,9 @@ def main():
print("-" * 20)

# Update gh metrics via api for all packages
print("Getting GitHub metrics for all packages...")
repo_endpoints = process_review.get_repo_endpoints(accepted_reviews)
all_reviews = process_review.get_gh_metrics(
repo_endpoints, accepted_reviews
)
# Contrib count is only available via rest api
repo_paths = process_review.get_repo_paths(accepted_reviews)
all_reviews = github_api.get_gh_metrics(repo_paths, accepted_reviews)

with open("all_reviews.pickle", "wb") as f:
pickle.dump(all_reviews, f)
Expand Down
Loading