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 typings for stats app #728

Merged
merged 4 commits into from Nov 26, 2018
Merged

Fix typings for stats app #728

merged 4 commits into from Nov 26, 2018

Conversation

wolfy1339
Copy link
Collaborator

Before the typings file would be empty, now it will contain the proper typings

@wolfy1339 wolfy1339 requested a review from a team as a code owner October 9, 2018 23:15
Copy link
Contributor

@hiimbex hiimbex left a comment

Choose a reason for hiding this comment

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

LGTM! I'm gonna wait on #704 to merge this

@hiimbex
Copy link
Contributor

hiimbex commented Oct 25, 2018

@wolfy1339 wanna pull in those changes and merge master for this?

@wolfy1339
Copy link
Collaborator Author

Sure thing.

@wolfy1339
Copy link
Collaborator Author

Done.

According to the Octokit types, the paginate() function's callback has an argument of type `Promise<AnyResponse>`, but it was being used as if it wasn't in a Promise
@wolfy1339
Copy link
Collaborator Author

@hiimbex Please re-review this as I added some fixes for some stuff that popped up in the Travis CI build log

@wolfy1339
Copy link
Collaborator Author

Just a gentle ping, so you can look at the new changes and get this merged

Copy link
Contributor

@tcbyrd tcbyrd left a comment

Choose a reason for hiding this comment

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

These changes look good from my end. I forgot we never went back and changed these from any

@wolfy1339
Copy link
Collaborator Author

Hey there. Is there anything holding back this PR?

@tcbyrd
Copy link
Contributor

tcbyrd commented Nov 26, 2018

@wolfy1339 Sorry for the delay on this. I fixed the conflicts we caused from some other PRs that landed. Merging now. Thanks for the contribution!

@tcbyrd tcbyrd merged commit 3b3e555 into probot:master Nov 26, 2018
@welcome
Copy link

welcome bot commented Nov 26, 2018

Thanks for your contribution to probot! 🎉
Congrats!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants