Skip to content

Commit

Permalink
fix(remove-app): Use App.name instead of App.repo for consistent App …
Browse files Browse the repository at this point in the history
…name
  • Loading branch information
gavindsouza committed May 19, 2022
1 parent 117d456 commit a3b859b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions bench/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ def get(self):

@step(title="Archiving App {repo}", success="App {repo} Archived")
def remove(self):
active_app_path = os.path.join("apps", self.repo)
active_app_path = os.path.join("apps", self.name)
archived_path = os.path.join("archived", "apps")
archived_name = get_available_folder_name(f"{self.repo}-{date.today()}", archived_path)
archived_app_path = os.path.join(archived_path, archived_name)
Expand Down Expand Up @@ -224,7 +224,7 @@ def install_resolved_apps(self, *args, **kwargs):

@step(title="Uninstalling App {repo}", success="App {repo} Uninstalled")
def uninstall(self):
self.bench.run(f"{self.bench.python} -m pip uninstall -y {self.repo}")
self.bench.run(f"{self.bench.python} -m pip uninstall -y {self.name}")

def _get_dependencies(self):
from bench.utils.app import get_required_deps, required_apps_from_hooks
Expand Down

0 comments on commit a3b859b

Please sign in to comment.