Skip to content

feat!: internally compute indexes at match time (removes get/set_index)#1001

Merged
LoricAndre merged 6 commits intomasterfrom
feat/better-indexes
Mar 10, 2026
Merged

feat!: internally compute indexes at match time (removes get/set_index)#1001
LoricAndre merged 6 commits intomasterfrom
feat/better-indexes

Conversation

@LoricAndre
Copy link
Contributor

@LoricAndre LoricAndre commented Mar 10, 2026

Checklist

  • The title of my PR follows conventional commits
  • I have updated the documentation (README.md, comments, src/manpage.rs and/or src/options.rs if applicable)
  • I have added unit tests
  • I have added integration tests
  • I have linked all related issues or PRs

Description of the changes

Note: codecov runs on the PR on this repo, but feel free to ignore it.

@codecov
Copy link

codecov bot commented Mar 10, 2026

Codecov Report

❌ Patch coverage is 56.00000% with 55 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/tui/event.rs 0.00% 19 Missing ⚠️
src/tui/app.rs 61.53% 7 Missing and 3 partials ⚠️
src/tmux.rs 22.22% 7 Missing ⚠️
src/helper/item.rs 0.00% 6 Missing ⚠️
src/tui/item_list.rs 53.84% 6 Missing ⚠️
src/item.rs 57.14% 3 Missing ⚠️
src/engine/fuzzy.rs 50.00% 1 Missing ⚠️
src/skim.rs 0.00% 1 Missing ⚠️
src/skim_item.rs 0.00% 1 Missing ⚠️
src/util.rs 96.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@LoricAndre LoricAndre merged commit ab514a5 into master Mar 10, 2026
14 checks passed
@LoricAndre LoricAndre deleted the feat/better-indexes branch March 10, 2026 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: Reverting to empty query does not reset sort order

1 participant