feat: added support for passing variables

[im-sampm]

This change adds basic support for passing --vars to Act.

I chose to model the change on the env implementation rather than event for simplicity, but I feel like vars, env and input could probably be refactored to have a common implementation.

I haven't created test cases for this change.

[shubhbapna]

LGTM! Yes I did write the ArgumentMap class to reduce duplicate code between act options like env, matrix, platforms etc but it mainly focuses on having no duplicate code for the toActArguments method. I am open to considering better design options!

[im-sampm]

[heart] Sam Peddamatham reacted to your message:

…

________________________________ From: Shubh Bapna ***@***.***> Sent: Tuesday, January 9, 2024 8:06:11 AM To: kiegroup/act-js ***@***.***> Cc: Sam Peddamatham ***@***.***>; Author ***@***.***> Subject: Re: [kiegroup/act-js] Added support for passing variables (PR #68) @shubhbapna approved this pull request. LGTM! Yes I did write the ArgumentMap class to reduce duplicate code between act options like env, matrix, platforms etc but it mainly focuses on having no duplicate code for the toActArguments method. I open to considering better design options! — Reply to this email directly, view it on GitHub<#68 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BBRCVFCHBRUAE5FSHAPOI2DYNT27HAVCNFSM6AAAAABBPDRODOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQMJQGU4TGOBYGQ>. You are receiving this because you authored the thread.Message ID: ***@***.***>