Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Multiselect flag support #882

Merged

Conversation

henriquemoraeszup
Copy link
Contributor

@henriquemoraeszup henriquemoraeszup commented Mar 4, 2021

Description

When adding a multiselect input, Ritchie could not translate it into flags. We need to take this case into account

  • Adds support to list input type command
  • Adds support to list input type input
  • Adds support to multiselect input type command
  • Adds support to multiselect input type input
  • Refactors tests on formula package
  • Makes a forward-compatible solution for flag reading by extracting any new types as string

Closes #881
Closes #791

How to verify it

Add a multiselect input and run --help, the input should be listed

Changelog

Supporting flags for multiselect input

Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
@henriquemoraeszup henriquemoraeszup added the 🪲 bug Report a bug encountered while operating Ritchie label Mar 4, 2021
@henriquemoraeszup henriquemoraeszup self-assigned this Mar 4, 2021
Copy link
Contributor

@GuillaumeFalourd GuillaumeFalourd left a comment

Choose a reason for hiding this comment

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

🚀

brunasilvazup
brunasilvazup previously approved these changes Mar 4, 2021
Copy link
Contributor

@brunasilvazup brunasilvazup left a comment

Choose a reason for hiding this comment

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

🔝

@brunasilvazup brunasilvazup added the ✔️ ready-for-review ready for review label Mar 4, 2021
@henriquemoraeszup
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Mar 4, 2021

🔥 Merge Conflict

@henriquemoraeszup
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Mar 4, 2021

👌 Merged branch hm/multiselect-flag-support into qa

Signed-off-by: Henrique Moraes <henrique.moraes@zup.com.br>
@henriquemoraeszup
Copy link
Contributor Author

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Mar 4, 2021

👌 Merged branch hm/multiselect-flag-support into qa

@henriquemoraeszup henriquemoraeszup merged commit aaea207 into ZupIT:master Mar 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🪲 bug Report a bug encountered while operating Ritchie ✔️ ready-for-review ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flags do not support multiselect Tests with Testify in Pkg/Formula
4 participants