Skip to content

Conversation

@ctcpip
Copy link
Contributor

@ctcpip ctcpip commented Aug 8, 2025

uses an octokit client that doesn't require auth

uses an octokit client that doesn't require auth
@ctcpip ctcpip mentioned this pull request Aug 8, 2025
Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM, After this is merged, I’ll continue with my PR.

@ctcpip
Copy link
Contributor Author

ctcpip commented Aug 8, 2025

@bjohansebas I pulled your commit into here

@ctcpip ctcpip force-pushed the issues-prs-regression branch 3 times, most recently from 1f3a415 to b9a6269 Compare August 8, 2025 18:18
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Co-authored-by: ctcpip <ctcpip@users.noreply.github.com>
@ctcpip ctcpip force-pushed the issues-prs-regression branch from b9a6269 to ae0d9f5 Compare August 8, 2025 18:20
@ctcpip ctcpip mentioned this pull request Aug 8, 2025
package.json Outdated
"@hackmd/api": "^2.5.0",
"ejs": "^3.1.10",
"@js-temporal/polyfill": "^0.5.1",
"@octokit/rest": "^22.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

This should be a dev dep.

@wesleytodd wesleytodd merged commit 1eee303 into pkgjs:main Aug 8, 2025
4 of 6 checks passed
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.

3 participants