Skip to content

Conversation

@barries
Copy link
Owner

@barries barries commented Jul 15, 2025

This PR addresses a very minor comment in the default config.toml, which says that the enter_accept config option "Defaults to true" when the atuin binary actually defaults to false if the option is not present / commented out.

That comment confused me (I commented out the seemingly unnecessary line) because most comments like that one—including throughout the rest of this file—use "default" to describe what the code does if the option is not present (i.e. if it's commented out in this file).

Happy to reword as needed (or, feel free). I'm sure it can be improved.

Checks

  • I am happy for maintainers to push small adjustments to this PR, to speed up the review cycle
  • I have checked that there are no existing pull requests for the same thing

The binary defaults to false for back compat but the comment implies that it defaults to true.

config.toml overrides the built-in default "false" by setting it to "true" to improve usability for new users.
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.

2 participants