Skip to content

fix: correct Linear API key settings URL - #134

Merged
bougyman merged 5 commits into
mainfrom
CRY-52-correct-api-key-settings-url
Aug 16, 2026
Merged

fix: correct Linear API key settings URL#134
bougyman merged 5 commits into
mainfrom
CRY-52-correct-api-key-settings-url

Conversation

@bougyman

Copy link
Copy Markdown
Member

Summary

  • Fixes the URL shown in the LINEAR_API_KEY is not set error message: https://linear.app/settings/apihttps://linear.app/settings/account/security
  • Updates the matching test assertion in missing_api_key_test.exs
  • Updates Readme.adoc setup instructions with the correct URL

Linear moved API key management from /settings/api to Settings > Account > Security & Access (/settings/account/security). The old URL was sending users to a non-existent page.

Closes CRY-52.

Test plan

  • mix test test/linear_cli/cli/missing_api_key_test.exs passes (verified locally)
  • Full test suite: 248/254 pass (6 pre-existing LinearCli.GitTest failures unrelated to this change)

🤖 Generated with Claude Code

The settings page for API keys moved from /settings/api to
/settings/account/security. Update the error message shown when
LINEAR_API_KEY is unset, its test assertion, and the project readme.

Fixes CRY-52.
Copilot AI lite review requested due to automatic review settings August 16, 2026 19:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The corrected Linear API key URL is longer than the old one, pushing
the IO.puts call past the formatter's line-length limit. Run mix format
to reformat the call across multiple lines.
@bougyman

Copy link
Copy Markdown
Member Author

Rework — formatter fix (Run 2)

Addressed the formatting issue raised in code review.

What changed:

  • app/lib/linear_cli/cli.ex:314 — ran mix format to split the IO.puts/2 call across multiple lines (the new URL is longer than the old one, exceeding the formatter's line-length limit)

Verification:

  • mix format --check-formatted — passes
  • mix test test/linear_cli/cli/missing_api_key_test.exs — 1/1 passes
  • No other files changed

@bougyman
bougyman merged commit 382e5cf into main Aug 16, 2026
2 checks passed
@bougyman
bougyman deleted the CRY-52-correct-api-key-settings-url branch August 16, 2026 19:47
This was referenced Aug 16, 2026
bougyman pushed a commit that referenced this pull request Aug 16, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.13.2](v1.13.1...v1.13.2)
(2026-08-16)


### Bug Fixes

* correct Linear API key settings URL
([#134](#134))
([382e5cf](382e5cf))
* removes tap
([#137](#137))
([9957242](9957242))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants