Repository navigation
Conversation
Keep indexed values consistent with iteration and preserve lazy evaluation. Cover tuple and named-tuple indexing, changed result types, and errors. AI disclosure: This work was prepared with assistance from OpenAI Codex.
Use scoped GitHub OIDC for repository branches and preserve the tokenless fork path. Keep the existing test matrix and coverage configuration. AI disclosure: This work was prepared with assistance from OpenAI Codex.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Indexing a mapped tuple or named tuple now applies the mapping function, matching iteration. For example,
mapview(x -> 10x, (1, 2, 3))[2]returns20instead of the original2. The shared indexing method preserves lazy evaluation and propagates errors from the parent or mapping function.Validation on Julia 1.10.12 and 1.13.1: the focused reproducer changed from 3 passing and 7 failing checks to all 10 passing. The full native suite passes all 224 assertions on both versions, including the new checks for indexing, changed output types, laziness, and error handling. All four hosted native test jobs pass at
c5e34de63add8bc179a9737ae0bddb92c629009c. All four coverage uploads are accepted for this repository and commit, and Codecov has processed all four reports (73.94%). An independent approving GitHub review is still required before merge.CI also authenticates coverage uploads with GitHub OIDC on repository branches and fails when an upload is rejected. The previous uploader rejected reports for the protected branch while still reporting a successful job.
Co-authored by Codex
AI disclosure: This work was prepared with assistance from OpenAI Codex.