[CI] Fix ESLint Workflow

[M-DEV-1]

Notes for Reviewers

• This PR fixes [CI] ESLint Checks are failing #39

Modified ESLint Workflow and made changes suggested by the linter.

Signed commits

• Yes, I signed my commits.

[fahdfady]

LGTM! it's a good note to use npm install here to ensure everything would work correctly

[M-DEV-1]

Yes, thanks for the input, @fahdfady.

I'm not entirely sure if NODE_ENV=test is strictly required since I'm running npm install to get all the project dependencies but I've still added it because of these errors:

0:0  error  Parsing error: [BABEL] /home/runner/work/shape-builder/shape-builder/site/src/components/Footer/index.js: 

babel-preset-gatsby has been loaded, which consumes config generated by the Gatsby CLI.

Set NODE_ENV=test to bypass, or run gatsby build first. 
 
(While processing: "/home/runner/work/shape-builder/shape-builder/site/node_modules/babel-preset-gatsby/index.js")

[leecalcote]

Merge conflict...

[M-DEV-1]

@leecalcote I resolved the merge conflicts, and the eslint-workflow seems to be working now.