#11 Implement Lint Verification in CI/CD

[cstroliadavis]

NOTE: This will not auto fix lint issues. After looking at the problem a bit, it seems clear that this should not be happening in a CI/CD process remotely, but would be better to do locally in a git hook.

Issue No:

closes: GH-11

Issue Type:

• [ ] Architecture (ensure that the architecture lead is included in the reviewers
• [ ] Bug
• [ ] Chore
• [ ] Clean-up
• [x] DevOps (ensure that the devops lead is included in the reviewers)
• [ ] Documentation
• [ ] Feature Request (ensure that the team lead and architecture lead are included in reviewers if this involves any core functionality changes)

Description

Adds lint checking to the PR pipeline

How Has This Been Tested?

Will test in this PR and an intentional fail PR #29.

Checklist:

• [x] This is not a code change, or my code follows the style guidelines of this project and I have run eslint
• [x] I have performed a self-review of my own changes
• [x] I have not added or modified any publicly available objects or methods or I have added appropriate jsdoc comments
• [x] I did not make any abnormal changes or I have added comments describing "why" I did things differently than usual.
• [x] This is a documentation update, or else I have made corresponding changes to the documentation
• [x] My changes generate no new errors or warnings
• [x] I have not added any code the requires unit test updates, or I have added unit tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes