Add ability to add collaborator by mtscout6 · Pull Request #462 · github-tools/github
| sudo: false | ||
| language: node_js | ||
| node_js: | ||
| - '8' |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, I opened PR #468 to to test against Node.js v8, which can be closed if this is merged.
It seems like this API was considered preview? Newer docs have the URL being /repos/:owner/:repo/collaborators/:username, which is the same as the one that was used, so that's nice!
This PR would be easier to review if it didn't also switch the CI and add async await. @clayreimann Is there any plans to change which versions of Node this library builds for? I'm thinking we could merge in the addCollaborator stuff and leave async await for another day.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters