Skip to content

bump version to v1.0.16#167

Merged
RyanLee-Dev merged 1 commit into
mainfrom
chore/bump-v1.0.16
Jun 1, 2026
Merged

bump version to v1.0.16#167
RyanLee-Dev merged 1 commit into
mainfrom
chore/bump-v1.0.16

Conversation

@RyanLee-Dev
Copy link
Copy Markdown
Collaborator

Summary

  • bump package.json version 1.0.151.0.16
  • expand README OAuth/API key auth section

@RyanLee-Dev RyanLee-Dev merged commit 109287d into main Jun 1, 2026
3 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

placeholder

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

Review: Update auth docs + version bump

Summary: README clarification on OAuth vs API-key auth priority, plus a patch version bump. Touched files: only docs and package.json.


Checks

Command Result Note
bun run typecheck ✅ Pass No type errors
bun test ✅ Pass 355 tests, 0 failures
bun run lint 🛑 Errors 2 errors, 12 warnings (pre-existing, see below)

Issues

None introduced by this PR. The diff is README + version bump only.

Pre-existing lint errors (not blocking this PR, but worth fixing)

File Issue
test/auth/oauth.test.ts:79 mockDeviceCodeFlow declared but never used
test/sdk/image.test.ts:4 ImageSaveOptions imported but never used
test/sdk/music.test.ts + 5 others @typescript-eslint/no-explicit-any warnings — should use unknown or a specific type

These errors predate this PR (unrelated files, no changes here). Suggest cleaning them up in a follow-up PR to keep lint green.


Verdict

🛑 Request Changes — lint is red due to pre-existing errors in test files. This PR itself introduces no issues, but the codebase cannot merge with lint failures.

Suggested fix: Either (a) suppress the two no-unused-vars errors with eslint-disable comments in those specific lines, or (b) remove the unused declarations. The any warnings are lower priority but should also be addressed before merging.

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.

1 participant