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

Policy: double quote: allow if dollar in string #340

Merged
merged 1 commit into from
Dec 24, 2019
Merged

Policy: double quote: allow if dollar in string #340

merged 1 commit into from
Dec 24, 2019

Conversation

tinmarino
Copy link
Contributor

One line to allow double quotes if there is a dollar in the string.
As example from my vimrc:

let $h = expand($HOME)
let $w = expand("$h/wiki/wiki") | let $wiki = $w

Cleaner than $h . '/wiki/wiki' in my opignon

@coveralls
Copy link

coveralls commented Dec 24, 2019

Coverage Status

Coverage decreased (-0.6%) to 95.452% when pulling 45951a5 on tinmarino:master into bcd6562 on Kuniwak:master.

@codecov-io
Copy link

Codecov Report

Merging #340 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #340   +/-   ##
=======================================
  Coverage   93.69%   93.69%           
=======================================
  Files         133      133           
  Lines        4155     4155           
  Branches      417      417           
=======================================
  Hits         3893     3893           
  Misses        167      167           
  Partials       95       95

@blueyed blueyed merged commit e44dedf into Vimjas:master Dec 24, 2019
@blueyed
Copy link
Member

blueyed commented Dec 24, 2019

Thank you!

@tinmarino
Copy link
Contributor Author

Well thank you. Awesome plugin (I'm using it with vim's ale) ! Fast merge, congrat.
Keep on linting !

@tyru
Copy link
Collaborator

tyru commented Dec 28, 2019

@tinmarino
I don't have opinion about this policy. Just I want to question: do you know
expand("$h/wiki/wiki") and expand('$h/wiki/wiki') are the same result?

@tyru
Copy link
Collaborator

tyru commented Dec 28, 2019

I don't have opinion about this policy

ah, wait.
this changes built-in (enabled by default) policy.

I think this change is unnecessary because above two expressions are evaluated to the same results.

@blueyed
Copy link
Member

blueyed commented Dec 28, 2019

True, missed that.
Since expand('$HOME') also works it should also prefer single quotes there then I guess.
@tinmarino are we missing something, or do you agree this should be reverted?

@tinmarino
Copy link
Contributor Author

Hi guys. Tk @tyru !!

Actually I was missing something. My PR is an error, my bad : the $ symbol do not get expanded even with double quote.

echo "$HOME"    " -> $HOME
echo '$HOME'    " -> $HOME

Sorry for that.

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

5 participants