Skip to content

feat: added new array params #156

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Apr 21, 2021
Merged

feat: added new array params #156

merged 7 commits into from
Apr 21, 2021

Conversation

@christyjacob4
Copy link
Contributor

@christyjacob4 christyjacob4 commented Apr 16, 2021 •
edited
Loading

  • Support for new CLI array params
  • Updated README and examples to use the new array syntax

eldadfux
eldadfux requested changes Apr 20, 2021
View reviewed changes
@eldadfux eldadfux requested a review from lohanidamodar April 20, 2021 06:07
lohanidamodar
lohanidamodar approved these changes Apr 20, 2021
View reviewed changes
Copy link
Member

@lohanidamodar lohanidamodar left a comment

Choose a reason for hiding this comment

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

Apart from the changes @eldadfux requested. Everything else LGTM.

@eldadfux eldadfux merged commit 3987782 into master Apr 21, 2021
@lohanidamodar lohanidamodar deleted the feat-cli-array-params branch June 5, 2022 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eldadfux eldadfux eldadfux approved these changes

@lohanidamodar lohanidamodar lohanidamodar approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@christyjacob4 @eldadfux @lohanidamodar