Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

Conversation

@bseyb
Copy link
Contributor

@bseyb bseyb commented Jun 15, 2017

When doing a local invoke and passing the 'path' parameter, a json string can be loaded from file. This will add the possibility to directly load a javascript object from file (for example: output copied from another action ran locally).

When doing a local invoke and passing the 'path' parameter, a json string can be loaded from file. This will add the possibility to directly load a javascript object from file (for example: output copied from another action ran locally).
@jthomas
Copy link
Contributor

jthomas commented Jun 15, 2017

The code looks good for this, thanks for raising it.

Could you also look at adding a unit test to demonstrate this works and ensure it doesn't get broken by any future changes?

@jthomas jthomas modified the milestone: 0.8 Release Jun 15, 2017
@jthomas jthomas merged commit b005d9f into serverless:master Jun 16, 2017
@jthomas
Copy link
Contributor

jthomas commented Jun 16, 2017

Awesome! Congrats on your first PR.
💯💯💯

Keep them coming 😎

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants