-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: restructure the implementation #1
Conversation
d9d99cb
to
6d9099d
Compare
6d9099d
to
6a13c57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be great to split up the index.js
into multiple files to make it more understandable/maintainable. This would also make it easier to add tests.
Nit: would also be easier to review the main changes if the eslint
changes were in their own commit.
6f80da5
to
2184c4b
Compare
2184c4b
to
cfbd04f
Compare
Thanks for the feedback, guys @evanlucas @matt-march! I have decided to make changes one-by-one as listed here |
cfbd04f
to
86f4799
Compare
c1455c3
to
29cd013
Compare
e5ababb
to
55f81d0
Compare
55f81d0
to
1448432
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samir, all done, take what you like and reject the rest. Let me know if you have any questions!
1448432
to
7771f83
Compare
7771f83
to
999096a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a few small comments on grammar
Include the same `CONTRIBUTING.md` and `CODE_OF_CONDUCT.md` we have been using for other repos. Ref: #8 Semver: patch
999096a
to
3dd200c
Compare
Resolved all; I think, it's good to go, right, @mdeltito @alfeng6 @tanberry @darinspivey? |
🎉 This PR is included in version 2.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
refactor: restructure the implementation
This commit is also introducing huge data handling, which is needed mainly for CloudTrail.
Ref: #2
Semver: major
test: add unit test coverage
Initializing TAP and adding unit test cases to cover all the cases
Ref: #3
Semver: patch
feat(config): use @logdna/env-config for configuration
Replace current configuration with @logdna/env-logdna
Ref: #2
Semver: major
refactor(logger): introduce @logdna/logger as a logger
Replacing the current flushing mechanism with @logdna/logger
Ref: #2
Semver: minor
docs: update README and Contribution Guide and add Code of Conduct
Include the same
CONTRIBUTING.md
andCODE_OF_CONDUCT.md
we have been using for other repos.Ref: #8
Semver: patch