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

Possible fix for #416 #449

Merged
merged 1 commit into from
Jun 22, 2019
Merged

Possible fix for #416 #449

merged 1 commit into from
Jun 22, 2019

Conversation

DeMol-EE
Copy link
Contributor

In reply to #416 (comment), this PR contains the following changes to README.md:

  • Added a note for iTerm2 users under readme/configuration/highlight style.
  • Changed the link to the issue list under troubleshooting so that the default filters (only open issues) are not applied.

* Added a note for iTerm2 users under readme/configuration/highlight
style.

* Changed the link to the issue list under troubleshooting so that the
default filters (only open issues) are not applied.
@@ -106,8 +108,7 @@ bindkey '^ ' autosuggest-accept

## Troubleshooting

If you have a problem, please search through [the list of issues on GitHub](https://github.com/zsh-users/zsh-autosuggestions/issues) to see if someone else has already reported it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This whitespace is actually intentional, can update the commit to retain it?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, you know what, the whitespace is already inconsistent in this file. Gonna go ahead and merge, and I'll clean up some time in the future.

@ericfreese ericfreese merged commit ec43fcf into zsh-users:master Jun 22, 2019
@ericfreese
Copy link
Member

Thanks @Warkst!

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