bcgov / bcgov/nr-apm-stack

event stream processing - Technical debt

Open
#25 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
9
Forks
6
PR merge metrics
No merged PRs in 30d

Description

- [x] Implement eslint
- [x] Triage linting issues (Disable lint tests that can't be quickly fixed)
- [x] Follow LIFT principles for directory structure https://angular.io/guide/styleguide#lift
- [x] Filename convention should be consistent, avoid abbreviations, follow language conventions like not using the prefix i
- [x] Class name should match filename and should flow from most to least specific Example: ParserApacheImpl -> ApacheParser (file: apache.parser.ts)
- [ ] Enable all lint tests (Fix code style issues and enable -- should be broken out to individual rules)
- [x] Unit tests should avoid using injection

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.