Skip to content

Use single quotes for better compatibility#575

Merged
MrRio merged 1 commit intovercel:masterfrom
albinekb:editor-fish
Aug 6, 2016
Merged

Use single quotes for better compatibility#575
MrRio merged 1 commit intovercel:masterfrom
albinekb:editor-fish

Conversation

@albinekb
Copy link
Contributor

@albinekb albinekb commented Aug 6, 2016

When attempting to open preferences using ⌘+, when using fish, i get the following error

❯  bash -c "exec env ${EDITOR:=nano} ~/.hyperterm.js"
${ is not a valid variable in fish.
fish:  bash -c "exec env ${EDITOR:=nano} ~/.hyperterm.js"

That's because how fish handles single vs double quotes

This PR makes opening preferences work as expected with fish, without changing the behaviour when using bash, i haven't tested it using zsh

@MrRio
Copy link
Contributor

MrRio commented Aug 6, 2016

Works with zsh!

screen shot 2016-08-06 at 09 44 00

@MrRio MrRio merged commit cc112b6 into vercel:master Aug 6, 2016
chabou added a commit to chabou/hyper that referenced this pull request Aug 19, 2016
* master:
  chore(package): update react to version 15.3.1 (vercel#637)
  Fix vercel#527: validate cursorColor value and apply default if it fails (vercel#590)
  Added customChildrenBefore to the tabs. (vercel#580)
  Fix for markdown files (vercel#618)
  Provide clear selection of text in terminal view (vercel#608)
  Added shellArgs to the config. (vercel#572)
  Fix international tilde character, and ` and ´ (vercel#584)
  chore(package): update electron-prebuilt to version 1.3.3 (vercel#604)
  chore(package): update should to version 11.0.0 (vercel#602)
  Comments for ignored stuff
  A little shorter
  Unneeded space
  Open new window offset from last focused window (Credit: albinekb) (vercel#581)
  Fix maximizing behaviour (vercel#176)
  Fix mapXDispatch and allow plugin to access onWheel (credit: lkzhao) (vercel#578)
  Use single quotes for better compatibility (vercel#575)
  Add config for bell (vercel#468)
  chore(package): update electron-prebuilt to version 1.3.2 (vercel#553)
  Moved "file-uri-to-path" dep to app package.json (vercel#569)
@timothyis timothyis added this to the v0.8.0 milestone Aug 29, 2016
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.

3 participants