Skip to content

[dist][lint] Update to probot@9, jest@24, standard@12; fix lint issue #21

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

Merged
merged 1 commit into from
Dec 3, 2019

Conversation

decompil3d
Copy link
Contributor

I've built a lambda using this + probot@9 and all seems to work fine. Tests pass with this update as well.

Let me know if I can change anything to help this move forward. Thanks!

Closes #20

@decompil3d
Copy link
Contributor Author

@tcbyrd @JasonEtco Hoping to get your review on this. Should be a quick one. Thanks!

@Mihailoff
Copy link

Please merge

@decompil3d
Copy link
Contributor Author

@Mihailoff All up to date now

@decompil3d
Copy link
Contributor Author

@tcbyrd @JasonEtco Ping -- any chance we can get this merged?

Copy link
Member

@JasonEtco JasonEtco left a comment

Choose a reason for hiding this comment

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

Sorry for the wait - looks good, thanks for the PR!

@JasonEtco JasonEtco merged commit 878450d into probot:master Dec 3, 2019
@decompil3d decompil3d deleted the probot9 branch December 3, 2019 19:46
@decompil3d
Copy link
Contributor Author

@JasonEtco Are you going to handle the version bump & publish, or would you like me to open a PR for that?

@JasonEtco
Copy link
Member

JasonEtco commented Dec 3, 2019

Done 👍 just wanted to get #30 out as well. Published as 0.4.0: https://www.npmjs.com/package/@probot/serverless-lambda/v/0.4.0

npm
An extension for running Probot in AWS Lambda

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.

dependency on probot is pinned to 7.x.x
3 participants