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.

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

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

Merged
merged 3 commits into from
Jan 18, 2024

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 <[email protected]>
@vladdoster vladdoster self-assigned this Jan 14, 2024
@vladdoster vladdoster merged commit 3f4b685 into main Jan 18, 2024
4 checks passed
@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))
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