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

Full width select #1403

Merged
merged 10 commits into from
Nov 4, 2021
Merged

Full width select #1403

merged 10 commits into from
Nov 4, 2021

Conversation

TC72
Copy link
Contributor

@TC72 TC72 commented Nov 1, 2021

New PR.

@fdschmidt93
Copy link
Member

fdschmidt93 commented Nov 1, 2021

Thanks taking another crack at this :)

I think what Conni meant with version bump entails:

E: Thinking about it, maybe that's split out better into two PRs, one with version bump with breaking git commit message and the full width selection.

@Conni2461
Copy link
Member

We can also do 1 PR with 2 commits and we dont squash

@fdschmidt93
Copy link
Member

Ah yes, simple solutions 😆 😅

@TC72
Copy link
Contributor Author

TC72 commented Nov 2, 2021

Looks like the 0.5.1 requirement is making a test fail, I'll check.

Edit: yes, the test is using neovim 0.5.0m same for failing mac os test.

Run sudo apt-get update
  sudo apt-get update
  sudo apt-get install -y fd-find
  test -d _neovim || {
    mkdir -p _neovim
    curl -sL https://github.com/neovim/neovim/releases/download/v0.5.0/nvim-linux64.tar.gz | tar xzf - --strip-components=1 -C "${PWD}/_neovim"
  }
  mkdir -p ~/.local/share/nvim/site/pack/vendor/start
  git clone --depth 1 https://github.com/nvim-lua/plenary.nvim ~/.local/share/nvim/site/pack/vendor/start/plenary.nvim
  git clone --depth 1 https://github.com/kyazdani42/nvim-web-devicons ~/.local/share/nvim/site/pack/vendor/start/nvim-web-devicons
  ln -s $(pwd) ~/.local/share/nvim/site/pack/vendor/start
  shell: /usr/bin/bash -e {0}

Also from Conni's note, should I have done a commit where I explicitly say "Update plugin requirement to nvim 0.5.1"?

@fdschmidt93
Copy link
Member

Also from Conni's note, should I have done a commit where I explicitly say "Update plugin requirement to nvim 0.5.1"?

Yes, two commits (ideally not but not necessarily in order):

  1. BREAKING: bump minimum nvim version to 0.5.1 (then e.g. Packer highlights these)
  2. feat: full-width hl of selection

I realize that requires a bit of git-gymnastics; that's why I thought it's just suggested to bump in a separate PR.

Not sure, but we might want to have a sticky issue with breaking changes as well.

@TC72
Copy link
Contributor Author

TC72 commented Nov 2, 2021

OK, I removed the version bump from this PR and created this one: #1405

@Conni2461 Conni2461 merged commit 6652d4d into nvim-telescope:master Nov 4, 2021
@Conni2461
Copy link
Member

Thanks :)

FWIW the "git-gymnastics" should have been the easier route. Less branch switching, etc. 😆

razak17 pushed a commit to razak17/telescope.nvim that referenced this pull request Feb 1, 2022
Can be changed back by setting 
telescope.setup {
  defaults = {
    hl_result_eol = false,
  }
}
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.

None yet

3 participants