Skip to content

Add At and AtKV index-based lookup functions #minor#6

Merged
freeformz merged 3 commits intomainfrom
ffz/At
Feb 13, 2026
Merged

Add At and AtKV index-based lookup functions #minor#6
freeformz merged 3 commits intomainfrom
ffz/At

Conversation

@freeformz
Copy link
Owner

No description provided.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds two new index-based lookup functions (At and AtKV) to the seq library, following the established dual API pattern. These functions allow retrieving values by their 0-based position in a sequence, complementing the existing Find family of functions that search by value or predicate.

Changes:

  • Added At and AtKV functions for index-based lookup in sequences with proper error handling for negative and out-of-range indices
  • Added comprehensive example tests covering success, out-of-range, empty, and negative index cases
  • Updated README documentation to list the new functions under the "Finding" section

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
seq.go Implements At and AtKV functions with proper handling of edge cases (negative indices, out of range)
seq_test.go Adds example tests for both functions covering various scenarios
README.md Documents the new functions in the Search/Finding section
CLAUDE.md Adds documentation note about Go version changes requiring #minor tag (minor documentation improvement)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@freeformz freeformz enabled auto-merge (squash) February 13, 2026 19:56
@freeformz freeformz disabled auto-merge February 13, 2026 19:56
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@freeformz freeformz enabled auto-merge (squash) February 13, 2026 19:58
@freeformz freeformz merged commit 32d0fb1 into main Feb 13, 2026
9 checks passed
@freeformz freeformz deleted the ffz/At branch February 13, 2026 19:59
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