Skip to content

Conversation

lazysegtree
Copy link
Collaborator

@lazysegtree lazysegtree commented Jun 22, 2025

Summary by CodeRabbit

  • Tests
    • Expanded paste-operation tests with explicit clipboard state verification (cleared vs. retained) after each scenario, improving coverage and confidence in expected behavior.
  • Style
    • Cleaned up test utilities by removing an obsolete comment and trailing whitespace.
  • Chores
    • No user-facing changes. Functionality, performance, and public APIs remain unchanged.

This comment was marked as outdated.

@lazysegtree

This comment was marked as outdated.

@lazysegtree

This comment was marked as outdated.

This comment was marked as outdated.

@lazysegtree lazysegtree added pr needs work PR needs work before it can be further reviewed/merged test related PR / Issue related to testcases or testing in general. labels Jun 22, 2025
coderabbitai[bot]

This comment was marked as outdated.

coderabbitai[bot]

This comment was marked as outdated.

@lazysegtree lazysegtree marked this pull request as draft July 11, 2025 02:43
@lazysegtree lazysegtree marked this pull request as ready for review August 17, 2025 09:28
@lazysegtree lazysegtree changed the title Attempting to fix the clipboard clearing check in Paste test fix: Add clipboard clearing check in Paste test Aug 17, 2025
@lazysegtree lazysegtree requested a review from yorukot August 17, 2025 09:31
@lazysegtree lazysegtree changed the base branch from main to develop August 17, 2025 11:24
@lazysegtree lazysegtree merged commit f78507f into develop Aug 17, 2025
15 of 16 checks passed
yorukot pushed a commit that referenced this pull request Aug 18, 2025
…ate linter configs (#997)

PRs
- #1002
- #993
- #896

Issues
- #563
- #738

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **New Features**
* Added two configurable hotkeys (OpenCommandLine, OpenSPFPrompt) and a
new file preview panel with improved image/text previews and
resize-aware rendering.

* **Refactor**
* Integrated preview rendering into the app update flow for more
predictable, asynchronous preview updates and simplified rendering
paths.

* **Documentation**
* Reformatted many configuration and hotkey descriptions for clearer,
consistent presentation.

* **Tests**
* Enhanced paste-operation tests to assert clipboard state after paste.

* **Chores**
  * Adjusted per-file linting rules.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr needs work PR needs work before it can be further reviewed/merged test related PR / Issue related to testcases or testing in general.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant