Skip to content

♻️ refactor(columns): fetch column content using jq, not expr#99

Merged
jfbus merged 1 commit intomainfrom
drop_expr
Mar 5, 2026
Merged

♻️ refactor(columns): fetch column content using jq, not expr#99
jfbus merged 1 commit intomainfrom
drop_expr

Conversation

@jfbus
Copy link
Copy Markdown
Contributor

@jfbus jfbus commented Mar 5, 2026

Description

Column contents are now fetched using JQ instead of expr.

Goal:

  • standardize on a single query langage
  • reduce binary size by ~ 800KB

As | is a JQ construct, the syntax of the --columns flags changes to [+]<title>:<jq query for content>||<title>:<jq query for content>

Type of Change

Please check the relevant option(s):

  • 🐛 Bug fix
  • ✨ New feature
  • 🧹 Code cleanup or refactor
  • 📝 Documentation update
  • 🔧 Build or CI-related change
  • 🔒 Security fix
  • Other (specify):

How Has This Been Tested?

Please describe the test strategy:

  • Manual testing
  • Unit tests
  • Integration tests
  • Not tested yet

Checklist

  • I have followed the Contributing Guidelines
  • I have added tests or explained why they are not needed
  • I have updated relevant documentation (README, examples, etc.)
  • My changes follow the Conventional Commits specification
  • My commits include appropriate Gitmoji

Additional Context

@jfbus jfbus force-pushed the drop_expr branch 3 times, most recently from daa179f to 8261091 Compare March 5, 2026 13:23
@jfbus jfbus marked this pull request as ready for review March 5, 2026 13:45
@jfbus jfbus merged commit c3de66d into main Mar 5, 2026
4 checks passed
@jfbus jfbus deleted the drop_expr branch March 5, 2026 14:18
@jfbus jfbus added the kind/refactoring Refactoring or internal changes label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactoring Refactoring or internal changes

Development

Successfully merging this pull request may close these issues.

2 participants