Skip to content

fix: Say CONNECT_API_KEY is missing instead of erroring on NA (0.8.6) - #73

Merged
csgillespie merged 3 commits into
mainfrom
fix/71-missing-api-key
Oct 16, 2026
Merged

csgillespie merged 3 commits into
mainfrom
fix/71-missing-api-key

Conversation

@csgillespie

Copy link
Copy Markdown
Contributor

0.8.5 → 0.8.6

Closes #71

🤖 Generated with Claude Code

csgillespie and others added 2 commits October 16, 2026 13:35
set_key() called nchar() on the NA from an unset variable, so check()
failed with "missing value where TRUE/FALSE needed". It now rejects NA
before storing the value, and check_api_key() checks the key first.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BEZ4ZUv51gFQjAe1WePbJF
@csgillespie csgillespie self-assigned this Oct 16, 2026
@csgillespie
csgillespie merged commit 68f37ab into main Oct 16, 2026
5 checks passed
@csgillespie
csgillespie deleted the fix/71-missing-api-key branch October 16, 2026 12:53
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.

A missing CONNECT_API_KEY gives "missing value where TRUE/FALSE needed"

1 participant