Skip to content

fix: sort newest aliases newest first#1853

Open
JSap0914 wants to merge 1 commit into
eza-community:mainfrom
JSap0914:fix-sort-newest-alias
Open

fix: sort newest aliases newest first#1853
JSap0914 wants to merge 1 commit into
eza-community:mainfrom
JSap0914:fix-sort-newest-alias

Conversation

@JSap0914

@JSap0914 JSap0914 commented Jul 4, 2026

Copy link
Copy Markdown

Summary

  • Map --sort=new and --sort=newest to the existing newest-first modified-time ordering.
  • Map --sort=old and --sort=oldest to the existing oldest-first modified-time ordering.
  • Add parser regressions for the explicit new/newest/old/oldest aliases.

Fixes #1772

Tests

  • cargo fmt --all -- --check
  • cargo test deduce_sort_field_new --lib (failed before the fix, passed after the fix via focused alias tests)
  • cargo test deduce_sort_field_ --lib
  • cargo test options::filter::tests --lib
  • cargo test --test cli_tests cli_all_tests
  • cargo test --test cli_tests cli_unix_tests
  • cargo test --lib
  • cargo check --all-targets
  • Manual CLI QA with temp files using controlled mtimes: --sort=newest produced newest, middle, oldest; --sort=oldest produced oldest, middle, newest.

Copilot AI review requested due to automatic review settings July 4, 2026 21:47

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.

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.

Wrong order for --sort=newest?

2 participants