Skip to content

Use path-columns in result metadata hook#1244

Merged
fdncred merged 1 commit intonushell:mainfrom
ask4fusora:fix-metadata-path-columns
Apr 11, 2026
Merged

Use path-columns in result metadata hook#1244
fdncred merged 1 commit intonushell:mainfrom
ask4fusora:fix-metadata-path-columns

Conversation

@ask4fusora
Copy link
Copy Markdown
Contributor

Summary

  • replace deprecatedmetadata set --datasource-l withmetadata set --path-columns [name in the result module display hook
  • preserve path-aware rendering forl output while matching the current Nushell metadata API

Why

Nushell now warns that--datasource-l is deprecated and recommends--path-columns [name instead.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@fdncred fdncred merged commit 341154f into nushell:main Apr 11, 2026
1 check passed
@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Apr 11, 2026

Thanks

@Juhan280
Copy link
Copy Markdown
Contributor

Hi. Sorry this is not correct as $meta.source == ls is also deprecated (we don't have a way to emit warnings for accessing fields).

I will try to make a PR later

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.

3 participants