Skip to content

Conversation

@Uzlopak
Copy link
Contributor

@Uzlopak Uzlopak commented Nov 21, 2023

@gr2m
@wolfy1339

As requested here the "fixed" unit test. Because I bind the console methods very early, I needed to change the tests.

Resolves #ISSUE_NUMBER


Before the change?

After the change?

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@Uzlopak
Copy link
Contributor Author

Uzlopak commented Nov 21, 2023

@gr2m
Should I create an issue for this PR? Or will you merge it anyway? Is the policy very strict?

@wolfy1339
Copy link
Member

It isn't always necessary for an issue, especially when things are self explanatory. It's always better to make one especially for features

@wolfy1339 wolfy1339 added the Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR label Nov 21, 2023
@Uzlopak
Copy link
Contributor Author

Uzlopak commented Nov 22, 2023

@wolfy1339

can we merge this, and then make an overall update of all deps for probot?

@wolfy1339 wolfy1339 merged commit 9843e38 into octokit:main Nov 22, 2023
@github-actions
Copy link
Contributor

🎉 This PR is included in version 5.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Uzlopak Uzlopak deleted the follow-up branch November 22, 2023 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants