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

feat(run): allow passing Octokit and log #1984

Merged
merged 4 commits into from
Mar 12, 2024
Merged

Conversation

hkdobrev
Copy link
Contributor

@hkdobrev hkdobrev commented Mar 8, 2024

Resolves #1477

These are the only options from Probot which could not be set when using run.

Using the server directly doesn't allow you to automatically read options from .env.

run does that, but it doesn't allow you to set a custom Octokit constructor.

This PR teaches run to pass on more options to Probot.

  • Allow passing Octokit constructor to run
  • Allow passing Logger to run
  • Document the new options for run
  • Test the new options for run

@hkdobrev hkdobrev requested a review from a team as a code owner March 8, 2024 23:36
Copy link

welcome bot commented Mar 8, 2024

Thanks for opening this pull request! A contributor should be by to give feedback soon. In the meantime, please check out the contributing guidelines and explore other ways you can get involved.

@gr2m gr2m changed the title Allow passing Octokit and Logger to run feat(run): allow passing Octokit and log Mar 12, 2024
@gr2m gr2m merged commit d195264 into probot:master Mar 12, 2024
19 checks passed
Copy link

welcome bot commented Mar 12, 2024

Thanks for your contribution to probot! 🎉
Congrats!

Copy link

🎉 This PR is included in version 13.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Use custom Octokit constructor
3 participants