Remove unused dependencies #613

edlerd · 2024-01-19T14:47:48Z

Done

Run the LXD-UI:
- On the demo server via the link posted by @webteam-app below. This is only available for PRs created by collaborators of the repo. Ask @lorumic or @edlerd for access.
- With a local copy of this branch, run as described here.
Perform the following QA steps:
- ensure ci passes
- spot check demo server

webteam-app · 2024-01-19T14:47:52Z

Signed-off-by: David Edler <david.edler@canonical.com>

mas-who · 2024-01-23T09:00:09Z

LGTM. Let's first merge #621 then let CI run one more time before we merge this PR

edlerd · 2024-01-23T09:02:49Z

LGTM. Let's first merge #621 then let CI run one more time before we merge this PR

I'll merge this first, as #621 has conflicts anyhow that need to be resolved. Will rebase and check CI on #621

edlerd force-pushed the cleanup-deps branch 2 times, most recently from ecde6c0 to ec36f9d Compare January 19, 2024 15:06

edlerd added Review: Code needed Review: QA needed labels Jan 19, 2024

chore(deps) remove unused dependencies

a5c7bf0

Signed-off-by: David Edler <david.edler@canonical.com>

edlerd force-pushed the cleanup-deps branch from ec36f9d to a5c7bf0 Compare January 22, 2024 13:53

mas-who approved these changes Jan 23, 2024

View reviewed changes

mas-who added Review: Code +1 Review: QA +1 and removed Review: Code needed Review: QA needed labels Jan 23, 2024

edlerd merged commit 28aa0f5 into canonical:main Jan 23, 2024
6 checks passed

edlerd deleted the cleanup-deps branch January 23, 2024 09:03

github-actions bot pushed a commit that referenced this pull request Jan 23, 2024

workflow: remove all reports for PR #613

3313ed1