10BC0 feat: add support for custom history file path in config by kx0101 · Pull Request #17425 · nushell/nushell · GitHub
[go: up one dir, main page]
Skip to content

feat: add support for custom history file path in config #17425

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
kx0101 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: add support for custom history file path in config #17425

kx0101 wants to merge 1 commit into from

Conversation

@kx0101
Copy link
@kx0101 kx0101 commented Jan 25, 2026

Closes #17419

The PR adds support for configuring a custom history file path via $env.config.history.path in config.nu

We allow users to redirect history to /dev/null or any custom location, which is particularly useful for TTY login scenarios where the --no-history flag cannot be used

$env.config.history.path = "/dev/null"  # disable history
$env.config.history.path = "~/custom/history.txt"  # custom location

I checked:

  • builds successfully with cargo build --release
  • run toolkit check pr successfully
  • manual testing: Setting $env.config.history.path = "/dev/null" prevents history file creation
  • manual testing: Custom path works correctly

@github-actions
Copy link
github-actions bot commented Jan 25, 2026

Hey, just a bot checking in! You edited files related to the configuration.
If you changed any of the default values or added a new config option, don't forget to update the doc_config.nu which documents the options for our users including the defaults provided by the Rust implementation.
If you didn't make a change here, you can just ignore me.

@kx0101 kx0101 force-pushed the feat/add-history-path-config branch from a00b70a to d7ac869 Compare January 25, 2026 23:13
@loci-dev loci-dev mentioned this pull request Jan 25, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allow configuring history file path via config.nu

1 participant

@kx0101
0