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

Customizable multiselect printer #402

Merged
merged 3 commits into from Oct 1, 2022

Conversation

jochil
Copy link
Contributor

@jochil jochil commented Sep 9, 2022

Description

Adding options to the InteractiveMultiselectPrinter to customize it:

  • NoFilter to disable fuzzy search / filter
  • KeySelect and KeyConfirm to use custom keys for selecting an item and confirm selection

Scope

What is affected by this pull request?

  • Bug Fix
  • New Feature
  • Documentation
  • Other

Related Issue

Fixes #

To-Do Checklist

  • I tested my changes
  • I have commented every method that I created/changed
  • I updated the examples to fit with my changes
  • I have added tests for my newly created methods

@codecov
Copy link

codecov bot commented Sep 9, 2022

Codecov Report

Merging #402 (0250cff) into master (0da799e) will decrease coverage by 0.50%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #402      +/-   ##
==========================================
- Coverage   81.76%   81.25%   -0.51%     
==========================================
  Files          29       29              
  Lines        2084     2097      +13     
==========================================
  Hits         1704     1704              
- Misses        365      378      +13     
  Partials       15       15              
Impacted Files Coverage Δ
interactive_multiselect_printer.go 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jochil jochil force-pushed the customizable-multiselect-printer branch from 9712ff3 to b5668d5 Compare September 9, 2022 12:18
@MarvinJWendt
Copy link
Member

Thanks for the PR! I'll take a look soon 👍

Copy link
Member

@MarvinJWendt MarvinJWendt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a little typo here :)

interactive_multiselect_printer.go Outdated Show resolved Hide resolved
interactive_multiselect_printer.go Outdated Show resolved Hide resolved
interactive_multiselect_printer.go Outdated Show resolved Hide resolved
@MarvinJWendt
Copy link
Member

Hi @jochil, I have left some comments, the rest looks good! I'll merge it when you've fixed the comments 🚀

@jochil jochil force-pushed the customizable-multiselect-printer branch 3 times, most recently from 8913607 to 69698cb Compare September 12, 2022 15:07
@jochil
Copy link
Contributor Author

jochil commented Sep 12, 2022

Hi @jochil, I have left some comments, the rest looks good! I'll merge it when you've fixed the comments rocket

@MarvinJWendt Thanks for the feedback... I fixed the comments

@MarvinJWendt MarvinJWendt self-assigned this Sep 20, 2022
@jochil jochil force-pushed the customizable-multiselect-printer branch from 69698cb to 24d0594 Compare September 22, 2022 20:28
@jochil
Copy link
Contributor Author

jochil commented Sep 29, 2022

@MarvinJWendt Hi. Is there something else I can adjust, please let me know :)

@MarvinJWendt
Copy link
Member

Hi @jochil, sorry for the long wait, I will review it now :)

Copy link
Member

@MarvinJWendt MarvinJWendt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just renamed the example. Thanks for the PR, and sorry for the delay. LGTM! 🚀

@MarvinJWendt MarvinJWendt merged commit b14fbe3 into pterm:master Oct 1, 2022
renovate bot added a commit to defenseunicorns/zarf that referenced this pull request Oct 3, 2022
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/pterm/pterm](https://togithub.com/pterm/pterm) | require |
patch | `v0.12.47` -> `v0.12.48` |

---

### Release Notes

<details>
<summary>pterm/pterm</summary>

### [`v0.12.48`](https://togithub.com/pterm/pterm/releases/tag/v0.12.48)

[Compare
Source](https://togithub.com/pterm/pterm/compare/v0.12.47...v0.12.48)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Exciting New Features 🎉

- Customizable multiselect printer by
[@&#8203;jochil](https://togithub.com/jochil) in
[pterm/pterm#402

**Full Changelog**:
pterm/pterm@v0.12.47...v0.12.48

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click
this checkbox.

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/defenseunicorns/zarf).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4yMDguMiIsInVwZGF0ZWRJblZlciI6IjMyLjIwOC4yIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Noxsios pushed a commit to defenseunicorns/zarf that referenced this pull request Mar 8, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/pterm/pterm](https://togithub.com/pterm/pterm) | require |
patch | `v0.12.47` -> `v0.12.48` |

---

### Release Notes

<details>
<summary>pterm/pterm</summary>

### [`v0.12.48`](https://togithub.com/pterm/pterm/releases/tag/v0.12.48)

[Compare
Source](https://togithub.com/pterm/pterm/compare/v0.12.47...v0.12.48)

<!-- Release notes generated using configuration in .github/release.yml
at master -->

#### What's Changed

##### Exciting New Features 🎉

- Customizable multiselect printer by
[@&#8203;jochil](https://togithub.com/jochil) in
[pterm/pterm#402

**Full Changelog**:
pterm/pterm@v0.12.47...v0.12.48

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click
this checkbox.

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://app.renovatebot.com/dashboard#github/defenseunicorns/zarf).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4yMDguMiIsInVwZGF0ZWRJblZlciI6IjMyLjIwOC4yIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

None yet

3 participants