Skip to content
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

Dependencies Update #144

Merged
merged 7 commits into from
Oct 9, 2022
Merged

Dependencies Update #144

merged 7 commits into from
Oct 9, 2022

Conversation

kinostl
Copy link
Contributor

@kinostl kinostl commented Oct 8, 2022

Updates the dependencies to their latest versions and changes the code a little bit to adapt to their new requirements. All tests pass. Swapped log out for npmlog. Added a custom implementation of the report system. Updated some tests to account for new changes. Added ioredis to account for the requirements of the latest connect-redis. Decided on ioredis via connect-redis documentaiton.

@kinostl kinostl mentioned this pull request Oct 8, 2022
@franciscop
Copy link
Owner

wow thanks a lot! the redis bit has actually given me trouble in a project, it seems this might be why! Everything look good here so I'm merging, but please give me a couple of days to test it out locally including the new redis imp!

@franciscop franciscop merged commit 64fe0c3 into franciscop:master Oct 9, 2022
@kinostl
Copy link
Contributor Author

kinostl commented Oct 9, 2022

Glad I could help out! fyi, npmlog is a bit chatty, but it was the only logging utility I could find that worked close enough to how log used to work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants