Fix typo in venv generic names variables

k-lyda commented 5 years ago

Description

I think this there is a leftover from migrating venv section from Spaceship prompt - name of the variable containing venv generic names is SPACESHIP_VENV_GENERIC_NAMES instead of SPACEFISH_VENV_GENERIC_NAMES

Motivation and Context

In the documentation name for this var is SPACEFISH_VENV_GENERIC_NAMES. If you add a line to config.fish according to the documentation, the generic names array is not updated, cause spaceship does not take var SPACEFISH_VENV_GENERIC_NAMES into account, it only looks for SPACESHIP_VENV_GENERIC_NAMES

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Screenshots (if appropriate):

How Has This Been Tested?

[x] I have tested using MacOS
[ ] I have tested using Linux

Checklist:

[x] I have checked that no other PR duplicates mine
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have updated the tests accordingly.

matchai commented 5 years ago

@all-contributors please add @k-lyda for bug

allcontributors[bot] commented 5 years ago

@matchai

I've put up a pull request to add @k-lyda! :tada:

matchai commented 5 years ago

:tada: This PR is included in version 2.3.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matchai / spacefish