cytoscape / ipycytoscape

A Cytoscape Jupyter widget
https://ipycytoscape.readthedocs.io/en/master/
BSD 3-Clause "New" or "Revised" License
264 stars 62 forks source link

fix quoting in test setup command #279

Closed dreness closed 2 years ago

dreness commented 2 years ago

The value used with the -e option to pip install is .[test]. The [ character has special meaning in many popular unix shells, so must be quoted (or escaped) to get the literal character needed for pip install to interpret the value inside the square brackets as a local directory path.

A demonstration using zsh:

% pip install -e .[test]
zsh: no matches found: .[test]

Quoting .[test] yields the expected result:

% pip install -e ".[test]"
Looking in indexes: ...

It seems the pypa documentation has the same problem - see pip install example #7 . Those examples are all prefixed with python -m pip, however the same shell quoting rules still apply. I'll file a similar PR over there.

github-actions[bot] commented 2 years ago

Binder :point_left: Launch a binder notebook on branch dreness/ipycytoscape/patch-1