Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

Use GitHub API instead of trying to run everything with local commands.#12

Merged
wlynch merged 4 commits intomainfrom
api
Mar 8, 2023
Merged

Use GitHub API instead of trying to run everything with local commands.#12
wlynch merged 4 commits intomainfrom
api

Conversation

@wlynch
Copy link
Member

@wlynch wlynch commented Mar 7, 2023

Local state gets a bit wonky if you aren't tracking a ref. Instead of trying to figure this out locally, switch the the GitHub API (which is likely closer to how this will need to be implemented in Scorecards anyway).

Fixes #11

@wlynch wlynch requested a review from cpanato March 7, 2023 21:54
Local state gets a bit wonky if you aren't tracking a ref. Instead of
trying to figure this out locally, switch the the GitHub API (which
is likely closer to how this will need to be implemented in Scorecards
anyway).
@wlynch wlynch merged commit 6a70ccc into main Mar 8, 2023
@wlynch wlynch deleted the api branch March 8, 2023 00:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Testing against non-tracked branches fails

2 participants

Comments