Skip to content

feat(delete): fix unsetting plugin state & add --debug / --quiet flags#622

Merged
vladdoster merged 3 commits intomainfrom
fix/delete-command
Jan 18, 2024
Merged

feat(delete): fix unsetting plugin state & add --debug / --quiet flags#622
vladdoster merged 3 commits intomainfrom
fix/delete-command

Conversation

@vladdoster
Copy link
Member

Description

Fix issue when deleting a plugin did not unset variables containing plugin state. This manifested in incorrect command output.

Add --debug && --quiet flags to delete command.

  • --debug flag enables xtrace for the currect delete command invocation
  • --quiet flag suppresses some output

Related Issue(s)

Motivation and Context

Screenshot 2024-01-13 at 23 23 15
Screenshot 2024-01-13 at 23 44 45

How Has This Been Tested?

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation change
  • New feature (non-breaking change which adds functionality)

Checklist:

  • All new and existing tests passed.
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.

Fix issue when deleting a plugin did not unset variables containing
plugin state. This manifested in incorrect command output.

Add `-d/--debug` && `-q/--quiet` flags to `delete` command.

* `--debug` flag enables xtrace for the currect delete command
  invocation
* `--quiet` flag suppresses some output

Signed-off-by: Vladislav Doster <mvdoster@gmail.com>
@vladdoster vladdoster self-assigned this Jan 14, 2024
@vladdoster vladdoster merged commit 3f4b685 into main Jan 18, 2024
@vladdoster vladdoster deleted the fix/delete-command branch January 18, 2024 19:40
github-actions bot pushed a commit that referenced this pull request Jan 18, 2024
# [3.13.0](v3.12.1...v3.13.0) (2024-01-18)

### Bug Fixes

* **ci:** remove duplicate gh-r  zunit job ([#623](#623)) ([70cefc0](70cefc0))
* debug logging logic and format ([#592](#592)) ([3c7e5be](3c7e5be))
* handle zinit --help ([#597](#597)) ([8cf9104](8cf9104))
* link ice check in .zinit-download-snippet function ([#608](#608)) ([358ef03](358ef03))
* pin dua gh-r test to v2.20.3 ([#605](#605)) ([0ba778a](0ba778a))
* silence unzip command in ziextract ([#614](#614)) ([71764bf](71764bf))
* tar permissions when `ziextract` ran as root user ([#557](#557)) ([e2d477c](e2d477c))
* update labeler config to v5 ([#604](#604)) ([794723c](794723c))

### Features

* configure, make, cmake, and build ices ([#613](#613)) ([#616](#616)) ([22e66db](22e66db))
* **delete:** fix unsetting plugin state & add `--debug` / `--quiet` flags ([#622](#622)) ([3f4b685](3f4b685))
* remove zpextract and use ziextract ([#611](#611)) ([2ccac85](2ccac85))
@github-actions
Copy link

🎉 This PR is included in version 3.13.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant