fix: Fix async main function type

[kammerjaeger]

This should better reflect the already allowed async function as main function in a probot app.

Let me know what you think.

[kammerjaeger]

Just a ping, what needs to be done to merge this?

[Uzlopak]

It seems correct:

probot/src/server/server.ts

Line 76 in 3219a98

await appFn(this.probotApp, {

@gr2m
@AaronDewes
target beta and merge?

[gr2m]

@gr2m @AaronDewes target beta and merge?

Why target beta? We do support it in current Probot, or am I missing something?

probot/src/server/server.ts

Lines 72 to 74 in 7b09369

	await appFn(this.probotApp, {
	getRouter: (path) => this.router(path),
	});

[Uzlopak]

@gr2m

Yes, main is valid target.

[AaronDewes]

@gr2m I can't merge this either, but I don't see any reason not to merge it.

[github-actions]

🎉 This PR is included in version 12.3.2 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 13.0.0-beta.9 🎉

The release is available on:

• GitHub release
• npm package (@beta dist-tag)

Your semantic-release bot 📦🚀