search

jhudsl / cow

A GitHub API wrapper in R. Has additional functions to help wrangle GitHub published courses like OTTR courses.
0 stars 0 forks source link

Add release info column and keywords and everything else #10

Closed cansavvy closed 2 years ago

cansavvy commented 2 years ago

Summary

Based on a conversation with @avahoffman and @carriewright11 today, we decided basing the readiness of a course on its release tag made the most sense.

But in order to act on this system we will need to extract release from the GitHub api. So that's what this PR is doing. Adding a new column that has the most recent release tag and release date.

Edit: I need to stop this PR before its becomes even more of a monstrosity.

Good news: This PR does put the package in a better place and will close #1 and #2 and addresses adding the releases, and keywords. It also neatens up some more things about how authorization is done. Bad news: that's too much for one PR. Whoops. I should have stopped this PR while I had the chance.

cansavvy commented 2 years ago

This PR has gotten a bit out of control and now also adds a keyword column. Meant to split this info separate PRs but I kept going anyway.

cansavvy commented 2 years ago

There’s still some lingering auth problems but this is already a huge pr so I’m gonna be a little reckless and just merge it. I’ll return to those auth issues on a separate PR. I know this runs fine locally.