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.

Sign up for GitHub

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

Jump to bottom

Update env remove logic #6195

Merged
merged 5 commits into from Sep 17, 2022
Merged

Update env remove logic #6195

merged 5 commits into from Sep 17, 2022

Commits on Aug 18, 2022

  1. Removed hardcoded project names for env testing 

    Added venv_name fixture to tests/utils
    @dhvcc
    dhvcc committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    04b0a92 View commit details
    Browse the repository at this point in the history

  2. Update full path handling for env remove 

    Add IncorrectEnvError exception
Add check_env_is_for_current_project method
Add a couple of tests for new and old env remove logic
    @dhvcc
    dhvcc committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    4d91dad View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2022

  1. Update env remove logic 

    Update remove handling, so that if other project's venv name is passed, raise IncorrectEnvError for better error message
    @dhvcc
    dhvcc committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    2700195 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into bug/6018

    @dhvcc
    dhvcc committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    a8aa448 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'master' into bug/6018

    @Secrus
    Secrus committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    1caf0e2 View commit details
    Browse the repository at this point in the history