Skip to content
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.

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

Make dir levels shown in the prompt configurable #26

Merged
merged 1 commit into from
Apr 16, 2019

Conversation

Ma27
Copy link
Contributor

@Ma27 Ma27 commented Apr 12, 2019

Until now the prompt only shows the last three levels of the prompt,
however I'd personally prefer to have some more levels (i.e. five).

I figured that it's not worth debating about how many levels make sense,
instead it's now configurable in ~/.zshrc, the current default will be
kept.

This can be achieved by adding the following line to ~/.zshrc before
loading this theme:

export LAMBDA_MOD_N_DIR_LEVELS=5

@halfo
Copy link
Owner

halfo commented Apr 16, 2019

Hey! Sorry for late reply. The PR looks great! As we are introducing a new config variable, do you think it would be helpful if we mention it in the README too?

Until now the prompt only shows the last three levels of the prompt,
however I'd personally prefer to have some more levels (i.e. five).

I figured that it's not worth debating about how many levels make sense,
instead it's now configurable in `~/.zshrc`, the current default will be
kept.

This can be achieved by adding the following line to `~/.zshrc` *before*
loading this theme:

```
export LAMBDA_MOD_N_DIR_LEVELS=5
```
@Ma27 Ma27 force-pushed the make-dir-levels-configurable branch from 54ef8a9 to 87e7cd7 Compare April 16, 2019 20:53
@Ma27
Copy link
Contributor Author

Ma27 commented Apr 16, 2019

You're absolutely right! Updated my commit and force-pushed :)

@halfo
Copy link
Owner

halfo commented Apr 16, 2019

Awesome!

@halfo halfo merged commit 6b83aed into halfo:master Apr 16, 2019
@Ma27 Ma27 deleted the make-dir-levels-configurable branch April 17, 2019 05:37
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