deepgram/deepgram-js-sdk

[Automation] Shellscript Linting

Closed this issue · 2 comments

Proposed changes

Shellscript Linting should adhere to known standards. Look into:

  • github.com/koalaman/shellcheck

This is already implemented in Go and needs to be ported over:
deepgram/deepgram-go-sdk#208

Context

Why is this change important to you? How would you use it? How can it benefit other users?

Possible Implementation

Not obligatory, but suggest an idea for implementing addition or change

Other information

Anything else we should know? (e.g. detailed explanation, related issues, links for us to have context, eg. stack overflow, codepen, etc)

We don't have any shellscripts :/

I will open a PR soon that will introduce some, but if we can add some of these things without shell scripts, we can close.

Reference to this:
#284