denox icon indicating copy to clipboard operation
denox copied to clipboard

WIP: Added support for inline scripting

Open wattry opened this issue 4 years ago • 3 comments

Description

Adding inline/Adhoc script support would allow devs to included much needs functionality seeing as no package manager is included, I believe this would enhance user experience at very little cost while allowing users to write short automation scripts.

I discovered this package while reading this article dev. I liked the package however it was not quite working how I needed it, nor how the OP was discussing.

Types of changes

  • New feature (non-breaking change which adds functionality)

Related links https://dev.to/vonheikemen/a-simple-way-to-replace-npm-scripts-in-deno-4j0g

Changelog

Added

  • replaceEnvVars + respective types and tests
  • _runInlineScript + respective types and tests
  • import dotenv

wattry avatar Jun 16 '20 21:06 wattry

@wattry Thank you for the contribution I quickly went through it and added a couple of comments.

I'm currently finishing environment variable support #25 I think this could be helpful for this PR.

Anything I could do to help with the PR?

BentoumiTech avatar Aug 11 '20 11:08 BentoumiTech

@wattry Thank you for the contribution I quickly went through it and added a couple of comments.

I'm currently finishing environment variable support #25 I think this could be helpful for this PR.

Anything I could do to help with the PR?

In my PR I implemented basic env support in my code so I would have to just remove that and wait on your env support to be released. As of now I don't think there is anything but if you have any suggestions I'm open to them

wattry avatar Aug 12 '20 13:08 wattry

I think it's okay, once I finish the env PR I'll merge it in your PR then release.

BentoumiTech avatar Aug 14 '20 02:08 BentoumiTech