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

the spread operator doesn't work with getters so we have to manually add it here (same for args) #513

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

xlmnxp
Copy link
Contributor

@xlmnxp xlmnxp commented Oct 1, 2022


fix bug when using tsyringe @autoInjectable() decorator the parser cannot read args from class

there 2 fixes for the issue, the first in the this PR and the second to pass args manually

const { args, flags } = await this.parse({...Connect, args: Connect.args });

maybe the fix is related to #603

@salesforce-cla
Copy link

salesforce-cla bot commented Oct 1, 2022

Thanks for the contribution! Before we can merge this, we need @xlmnxp to sign the Salesforce.com Contributor License Agreement.

@xlmnxp
Copy link
Contributor Author

xlmnxp commented Oct 1, 2022

Thanks for the contribution! Before we can merge this, we need @xlmnxp to sign the Salesforce.com Contributor License Agreement.

signed

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.

2 participants